HomePhabricator

Continue lifting column layout logic out of ColumnPositionQuery

Description

Continue lifting column layout logic out of ColumnPositionQuery

Summary:
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.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T10010

Differential Revision: https://secure.phabricator.com/D15175

Details

Provenance
epriestleyAuthored on Feb 3 2016, 6:58 PM
epriestleyPushed on Feb 3 2016, 11:05 PM
Reviewer
chad
Differential Revision
D15175: Continue lifting column layout logic out of ColumnPositionQuery
Parents
rP23b835b64731: Begin lifting column layout logic out of ColumnPositionQuery
Branches
Unknown
Tags
Unknown
Tasks
T10010: Implement Subprojects
Build Status
Buildable 10483
Build 12825: Run Core Tests