Page MenuHomePhabricator

Reduce drag-and-drop jank on dashboards
ClosedPublic

Authored by epriestley on Sat, Apr 13, 12:20 AM.

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
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

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