Page MenuHomePhabricator

Allow stacked comment actions to be explicitly ordered
ClosedPublic

Authored by epriestley on Apr 6 2016, 1:24 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, May 4, 12:16 AM
Unknown Object (File)
Fri, May 3, 10:03 AM
Unknown Object (File)
Thu, May 2, 7:23 PM
Unknown Object (File)
Sun, Apr 28, 10:35 AM
Unknown Object (File)
Thu, Apr 25, 3:29 AM
Unknown Object (File)
Wed, Apr 24, 3:28 AM
Unknown Object (File)
Fri, Apr 19, 2:26 AM
Unknown Object (File)
Fri, Apr 19, 2:26 AM
Subscribers
None

Details

Summary

Ref T6027. Normally, actions use the same order as the form, but in some cases (like moving stuff on workboards) it makes sense to reorder them explicitly.

Pin "Move on board" near the bottom, and "projects/subscribers" at the bottom. I think these are generally reasonable rules in all cases.

Test Plan

Opened menu, saw slightly better action order.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Allow stacked comment actions to be explicitly ordered.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Apr 6 2016, 4:07 PM
This revision was automatically updated to reflect the committed changes.