Page MenuHomePhabricator

Don't render a grip on tasks returned over AJAX to non-draggable UIs
ClosedPublic

Authored by epriestley on Aug 16 2014, 11:44 AM.
Tags
None
Referenced Files
F15285123: D10282.id24764.diff
Tue, Mar 4, 11:40 AM
Unknown Object (File)
Feb 7 2025, 10:35 PM
Unknown Object (File)
Feb 7 2025, 10:35 PM
Unknown Object (File)
Feb 7 2025, 10:35 PM
Unknown Object (File)
Feb 7 2025, 10:35 PM
Unknown Object (File)
Feb 3 2025, 2:10 AM
Unknown Object (File)
Feb 3 2025, 12:13 AM
Unknown Object (File)
Jan 31 2025, 9:22 PM
Subscribers

Details

Summary

Fixes T5140. When you ajax-edit a task and we send back a full-size card, we currently always put a drag grip on it.

If you clicked the "edit" thing from a priority-ordered list, this is appropriate. However, if you clicked it from some other type of list, it is not.

Pass the expected grippableness through the call.

Test Plan
  • Edited a task from a reorderable (priority-ordered) view, got grip.
  • Edited a task from a nonreorderable (author-ordered) view, got no grip.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Don't render a grip on tasks returned over AJAX to non-draggable UIs.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: chad, btrahan.
chad edited edge metadata.
This revision is now accepted and ready to land.Aug 16 2014, 3:02 PM
epriestley updated this revision to Diff 24764.

Closed by commit rP98a847a36c26 (authored by @epriestley).