Page MenuHomePhabricator

Add UI for alternate board ordering rules
ClosedPublic

Authored by epriestley on Aug 7 2014, 6:04 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, May 5, 4:03 AM
Unknown Object (File)
Fri, May 3, 9:01 AM
Unknown Object (File)
Mon, Apr 29, 4:03 AM
Unknown Object (File)
Fri, Apr 26, 3:40 PM
Unknown Object (File)
Fri, Apr 19, 2:07 AM
Unknown Object (File)
Fri, Apr 19, 2:07 AM
Unknown Object (File)
Fri, Apr 19, 2:07 AM
Unknown Object (File)
Fri, Apr 12, 3:18 AM

Details

Summary

Ref T4807. This doesn't actually do anything yet, but adds a dropdown menu for choosing an ordering and gets all the UI working correctly.

This also fixes a bug where column hidden state wouldn't persist across filter changes.

(I won't land this until it does something, but the next diff will probably be a mess so this seemed like a clean place to sever things.)

Test Plan

Screen_Shot_2014-08-07_at_10.55.04_AM.png (1×1 px, 178 KB)

  • Altered sort ordering.
  • Altered hidden state and filters, verified all states persisted correctly.
  • Added phlog() to edit/create and move controllers and verified they receive sort information.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Add UI for alternate board ordering rules.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.

Is this sticky per board or globally?

It only persists in the current browser window, like the filtering.

chad added a reviewer: chad.
This revision is now accepted and ready to land.Aug 7 2014, 7:41 PM

(I think making these sticky in some way would be reasonable, but would like to stabilize the functionality first. My gut is per-user-per-board? But this feels like it might be a can of worms and users haven't specifically asked after it yet.)

Yeah I've thought about this all morning and no clear answer. We can ship and get feedback from @swisspol, etc from actual use.

I think you have the right defaults though.

Yeah, my guess is that "Natural" will become 95% of use after it works.

(I think alternate sort orders are still useful, which is why I'm retaining "priority", and I'd like to pursue the headers-in-the-column design at some point to make priority more useful/explicit, but expect that to land somewhere over the horizon on the roadmap if Natural proves dominant.)

Maybe one sort could be "Bureaucracy" which tracks number of dependencies and manager weigh-ins.

epriestley updated this revision to Diff 24513.

Closed by commit rPfdf6b56261db (authored by @epriestley).