Page MenuHomePhabricator

Continue lifting column layout logic out of ColumnPositionQuery

Authored by epriestley on Feb 3 2016, 7:07 PM.
Referenced Files
F11145829: D15175.id36634.diff
Sat, Sep 24, 4:50 AM
Unknown Object (File)
Fri, Sep 16, 9:15 PM
Unknown Object (File)
Sat, Aug 27, 4:38 AM
Unknown Object (File)
Aug 18 2022, 5:08 PM
Unknown Object (File)
Aug 15 2022, 9:40 AM
Unknown Object (File)
Aug 15 2022, 5:18 AM
Unknown Object (File)
Aug 10 2022, 7:06 PM
Unknown Object (File)
Aug 7 2022, 12:49 AM



Ref T10010. See D15174. This gets rid of the "actually apply the change" callsite and moves it to layout engine.

Next up is to make the board view use the layout engine, then throw away all the whack code in ColumnPositionQuery, then move forward with D15171.

Test Plan
  • Dragged tasks within a column.
  • Dragged tasks between columns.
  • Dragged tasks to empty columns.
  • Created a task in a column.
  • Swapped board to priority sort, dragged a bunch of stuff all over.

Diff Detail

rP Phabricator
Lint Not Applicable
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Continue lifting column layout logic out of ColumnPositionQuery.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
  • Remove giant debugging block.
chad edited edge metadata.

Will all this stuff be testable?

This revision is now accepted and ready to land.Feb 3 2016, 7:50 PM

Yeah, that's one of the big advantages of doing it this way. A big part of this is ultimately UI stuff so there are limits to how much I can reasonably cover, but the basic "moving stuff around" operations should be fairly testable once I get the new thing in place.

This revision was automatically updated to reflect the committed changes.