Page MenuHomePhabricator

Reduce drag-and-drop jank on dashboards
ClosedPublic

Authored by epriestley on Apr 13 2019, 12:20 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Mar 13, 7:16 PM
Unknown Object (File)
Feb 12 2024, 12:35 PM
Unknown Object (File)
Feb 5 2024, 11:01 AM
Unknown Object (File)
Jan 25 2024, 2:02 AM
Unknown Object (File)
Jan 1 2024, 5:00 PM
Unknown Object (File)
Dec 28 2023, 5:09 PM
Unknown Object (File)
Dec 27 2023, 3:27 AM
Unknown Object (File)
Dec 24 2023, 4:54 PM
Subscribers
None

Details

Summary

Depends on D20414. Ref T13272. Several minor things here:

  • Currently, you can drag panels underneath the invisible "there are no items in this column" div and the "Create Panel / Add Existing Panel" buttons. This is silly; stop it.
  • Currently, when viewing a tab panel on a dashboard, you can drag the panels inside it. This is extremely silly. Make "movable" off by default and pass it through the async flow only when we actually need it.
  • Make the whole "Add Tab..." virtual tab clickable to open the dropdown. This removes the rare exception/todo combo I added earlier. F
  • Add or remove some icons or something.
Test Plan

Moved panels around on dashboards. Tried to drag panels inside tab panels. Added tab. Things were less obviously broken.

Diff Detail

Repository
rP Phabricator
Branch
panel17
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 22579
Build 30930: Run Core Tests
Build 30929: arc lint + arc unit

Event Timeline

epriestley retitled this revision from Reduce drag-and-drop jank on workboards to Reduce drag-and-drop jank on dashboards.Apr 13 2019, 12:26 AM
This revision is now accepted and ready to land.Apr 17 2019, 5:53 PM
This revision was automatically updated to reflect the committed changes.