Page MenuHomePhabricator

Make column reordering after edits on workboards more general
ClosedPublic

Authored by epriestley on Jul 5 2014, 7:24 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 25, 2:03 AM
Unknown Object (File)
Wed, Apr 24, 10:51 PM
Unknown Object (File)
Tue, Apr 23, 4:52 PM
Unknown Object (File)
Sat, Apr 20, 7:13 PM
Unknown Object (File)
Thu, Apr 11, 11:19 PM
Unknown Object (File)
Thu, Apr 11, 10:05 AM
Unknown Object (File)
Tue, Apr 2, 3:41 PM
Unknown Object (File)
Mar 25 2024, 6:21 AM
Subscribers

Details

Summary

Ref T5476. Currently, the task edit code assumes it knows what the UI looks like and sends back where on the column an item should be inserted.

This is buggy after adding filters, and relatively complex. Instead, send down the ordering on the whole column and sort it in the UI. This is a bit simpler overall and more general. It makes it easier to further generalize this code for T5476.

Test Plan
  • Edited a task on a board, changing priority. Saw it reorder properly.
  • Edited a task on a board in a field of other tasks at the same top-level priority. Saw it refresh without reordering.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Make column reordering after edits on workboards more general.
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.Jul 5 2014, 7:39 PM
epriestley updated this revision to Diff 23707.

Closed by commit rPfae23e086049 (authored by @epriestley).