Page MenuHomePhabricator

Remove the ability to drag tasks up and down on (non-Workboard) priority list views
ClosedPublic

Authored by epriestley on Mar 7 2019, 7:41 PM.
Tags
None
Referenced Files
F14003416: D20263.id48357.diff
Sat, Oct 26, 5:59 AM
F14000818: D20263.diff
Fri, Oct 25, 1:18 AM
F13980949: D20263.id.diff
Sat, Oct 19, 1:31 PM
F13974344: D20263.id48399.diff
Fri, Oct 18, 5:11 AM
F13974046: D20263.id48378.diff
Fri, Oct 18, 3:49 AM
F13973829: D20263.diff
Fri, Oct 18, 2:53 AM
Unknown Object (File)
Oct 2 2024, 6:06 AM
Unknown Object (File)
Sep 21 2024, 3:25 AM
Subscribers
None

Details

Summary

Ref T13074. Today, in normal task list views in Maniphest (not workboards), you can (sometimes) reorder tasks if the view is priority-sorted.

I suspect no one ever does this, few users know it's supported, and that it was basically rendered obsolete the day we shipped workboards.

This also means that we need to maintain a global "subpriority" for tasks, which distinguishes between different tasks at the same priority level (e.g., "High") and maintains a consistent ordering on workboards.

As we move toward making workboards more flexible (e.g., group by author / owner / custom fields), I'd like to try moving away from "subpriority" and possibly removing it entirely, in favor of "natural order", which basically means "we kind of remember where you put the card and it works a bit like a sticky note".

Currently, the "natural order" and "subpriority" systems are sort of similar but also sort of in conflict, and the "subpriority" system can't really be extended while the "natural order / column position" system can.

The only real reason to have a global "subpriority" is to support the list-view drag-and-drop.

It's possible I'm wrong about this and a bunch of users love this feature, but we can re-evaluate if we get feedback in this vein.

(This just removes UI, the actual subpriority system is still intact and still used on workboards.)

Test Plan

Viewed task lists, was no longer able to drag stuff. Grepped for affected symbols. Dragged stuff in remaining grippable lists, like "Edit Forms" in EditEngine config.

Diff Detail

Repository
rP Phabricator
Branch
grip1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 22216
Build 30377: Run Core Tests
Build 30376: arc lint + arc unit

Event Timeline

src/view/phui/PHUIObjectItemView.php
428

I am only a tiny bit sad to lose this bit of silliness.

  • Clean up one more piece of code I dug out of the ground.
This revision is now accepted and ready to land.Mar 12 2019, 6:37 PM
This revision was automatically updated to reflect the committed changes.