Page MenuHomePhabricator
Differential D13487

[Redesign] Fix Maniphest subpriority editing
ClosedPublic
Actions

Authored by chad on Jun 30 2015, 4:32 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Feb 8, 5:05 PM
Unknown Object (File)
Sat, Feb 8, 5:04 PM
Unknown Object (File)
Sat, Feb 8, 5:04 PM
Unknown Object (File)
Sat, Feb 8, 10:04 AM
Unknown Object (File)
Wed, Feb 5, 12:02 PM
Unknown Object (File)
Fri, Jan 31, 4:21 AM
Unknown Object (File)
Wed, Jan 29, 4:06 AM
Unknown Object (File)
Fri, Jan 24, 6:58 AM
View All Files
Subscribers
epriestley
• GoogleLegacy
Korvin
Tokens
"Piece of Eight" token, awarded by btrahan.

Details

Reviewers
epriestley
btrahan
Maniphest Tasks
T8099: Mid-2015 Phabricator Redesign
T8338: Unable to re-arrange Maniphest tasks into 1st row when dragging (redesign branch)
Commits
Restricted Diffusion Commit
rP2c922ca2306f: [Redesign] Fix Maniphest subpriority editing
Summary

Ref T8099, Fixes T8338. This allows re-ordering of Maniphest Tasks in the redesign. Somehow seems more fragile, but I couldn't break anything with it.

Test Plan

Try ordering into first position after header, last position, changing priority outright, everything I can drag.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad updated this revision to Diff 32629.Jun 30 2015, 4:32 PM
chad retitled this revision from to [Redesign] Fix Maniphest subpriority editing.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, btrahan.
chad added tasks: T8338: Unable to re-arrange Maniphest tasks into 1st row when dragging (redesign branch), T8099: Mid-2015 Phabricator Redesign.
Herald added subscribers: Korvin, epriestley. · View Herald TranscriptJun 30 2015, 4:32 PM
epriestley accepted this revision.Jun 30 2015, 4:36 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jun 30 2015, 4:36 PM
Closed by commit rP2c922ca2306f: [Redesign] Fix Maniphest subpriority editing (authored by chad, committed by chad). · Explain WhyJun 30 2015, 4:37 PM
This revision was automatically updated to reflect the committed changes.
btrahan awarded a token.Jun 30 2015, 6:14 PM
• GoogleLegacy added a subscriber: • GoogleLegacy.Sep 10 2018, 9:36 AM
This comment was removed by epriestley.
• GoogleLegacy added a comment.Sep 10 2018, 9:37 AM
This comment was removed by epriestley.

Revision Contents
Changeset List

PathSize
resources/
celerity/
20 lines
src/
applications/
maniphest/
view/
2 lines
webroot/
rsrc/
js/
application/
maniphest/
6 lines

Diff 32630

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/maniphest/view/ManiphestTaskResultListView.php

Loading...
View Options

webroot/rsrc/js/application/maniphest/behavior-subpriorityeditor.js

Loading...
Phabricator · Built by Phacility