Page MenuHomePhabricator
Differential D15638

Implement a rough optgroup-based "Move on Workboard" stacked action
ClosedPublic
Actions

Authored by epriestley on Apr 6 2016, 1:08 PM.
Tags
None
Referenced Files
F19929613: D15638.id.diff
Wed, Apr 8, 5:22 PM
F19898850: D15638.id37685.diff
Sun, Mar 29, 4:19 PM
F19889489: D15638.id37685.diff
Tue, Mar 24, 2:57 PM
F19889415: D15638.id37685.diff
Tue, Mar 24, 1:28 PM
F19809100: D15638.id37677.diff
Mar 3 2026, 8:07 AM
F19548957: D15638.diff
Jan 27 2026, 12:47 AM
F19510291: D15638.id37685.diff
Jan 11 2026, 6:57 AM
F19074565: D15638.id37680.diff
Dec 1 2025, 5:54 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T6027: Way to edit which column a task is in from the task itself
Commits
rP67629aab14b3: Implement a rough optgroup-based "Move on Workboard" stacked action
Summary

Ref T6027. Try this out and see how it feels? Clear issues:

  • This definitely shouldn't be at the top.
  • You should probably be able to select it multiple times?
  • Some of the "which columns show up" rules might need adjustment?
  • Diamond marker maybe not great?

Not sure I love this but it doesn't feel terrible...

Test Plan

Screen Shot 2016-04-06 at 5.58.31 AM.png (492×1 px, 59 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 37677.Apr 6 2016, 1:08 PM
epriestley retitled this revision from to Implement a rough optgroup-based "Move on Workboard" stacked action.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T6027: Way to edit which column a task is in from the task itself.
chad added inline comments.Apr 6 2016, 4:02 PM
src/applications/maniphest/editor/ManiphestEditEngine.php
426

yeah maybe just a checkmark?

epriestley added a comment.Apr 6 2016, 4:06 PM

Here's a checkmark version -- maybe better? Still seems a little weird to me.

Screen Shot 2016-04-06 at 9.04.26 AM.png (239×303 px, 25 KB)

Or we could do something like this?

Screen Shot 2016-04-06 at 9.06.09 AM.png (246×439 px, 27 KB)

chad accepted this revision.Apr 6 2016, 4:08 PM
chad edited edge metadata.

yeah lets start with the second one

This revision is now accepted and ready to land.Apr 6 2016, 4:08 PM
epriestley added a comment.Apr 6 2016, 4:10 PM

Sorry -- is the second one the checkmarks (the "second proposed approach") or "o•" (the "second screenshot in the comment")? :3

today is not a good brain day either for me tbqh

chad added a comment.Apr 6 2016, 4:11 PM

pasted_file (188×263 px, 18 KB)

epriestley added a comment.Apr 6 2016, 4:12 PM

cool I like this one (this one) too I think

Screen Shot 2016-04-06 at 9.11.50 AM.png (269×326 px, 41 KB)

that one

epriestley updated this revision to Diff 37680.Apr 6 2016, 4:12 PM
epriestley edited edge metadata.
  • Use empty and filled circles to show current column instead of diamonds.
Closed by commit rP67629aab14b3: Implement a rough optgroup-based "Move on Workboard" stacked action (authored by epriestley, committed by epriestley). · Explain WhyApr 6 2016, 4:15 PM
This revision was automatically updated to reflect the committed changes.
epriestley mentioned this in T6027: Way to edit which column a task is in from the task itself.Apr 6 2016, 4:26 PM

Revision Contents
Changeset List

PathSize
resources/
celerity/
12 lines
src/
2 lines
applications/
maniphest/
editor/
81 lines
transactions/
commentaction/
27 lines
editfield/
21 lines
webroot/
rsrc/
js/
phuix/
35 lines

Diff 37685

View Options

resources/celerity/map.php

Loading...
View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/maniphest/editor/ManiphestEditEngine.php

Loading...
View Options

src/applications/transactions/commentaction/PhabricatorEditEngineColumnsCommentAction.php

Loading...
View Options

src/applications/transactions/editfield/PhabricatorColumnsEditField.php

Loading...
View Options

webroot/rsrc/js/phuix/PHUIXFormControl.js

Loading...
Phabricator · Built by Phacility