Page MenuHomePhabricator

Basic stacked action support for EditEngine
ClosedPublic

Authored by epriestley on Dec 3 2015, 7:38 PM.
Tags
None
Referenced Files
F14039938: D14653.id35450.diff
Mon, Nov 11, 6:49 AM
F14012150: D14653.id35448.diff
Fri, Nov 1, 7:26 AM
F14012146: D14653.id35450.diff
Fri, Nov 1, 7:25 AM
F14012055: D14653.id.diff
Fri, Nov 1, 7:07 AM
F14011759: D14653.diff
Fri, Nov 1, 5:13 AM
F14004854: D14653.diff
Sun, Oct 27, 3:51 AM
F13972965: D14653.id35450.diff
Oct 17 2024, 10:06 PM
F13966673: D14653.id.diff
Oct 16 2024, 9:04 AM
Subscribers
None
Tokens
"Love" token, awarded by ftdysa."Mountain of Wealth" token, awarded by cburroughs.

Details

Summary

Ref T9132. This still has a lot of rough edges but the basics seem to work OK.

Test Plan

Screen Shot 2015-12-03 at 11.37.15 AM.png (1×1 px, 187 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Basic stacked action support for EditEngine.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
_______ < omg > ------- \ ^__^ \ (oo)\_______ (__)\ )\/\ ||----w | || ||
_____________________ < this is amooozing > --------------------- \ ^__^ \ (oo)\_______ (__)\ )\/\ ||----w | || ||
chad edited edge metadata.
This revision is now accepted and ready to land.Dec 3 2015, 7:52 PM

I played with this locally and nothing broke!

This definitely needs some more work (drafts don't save the actions, for example, and I have no idea what it does on mobile but probably not anything great, and there are more than a few very rough patches in this code) but it at least mostly works, and I don't think it regresses or breaks anything -- it may just not work as well as it should in some cases.

I think this is the last major piece we need before I can at least try bringing this to Maniphest. Not sure how well that'll go, but hopefully not too bad.

(One other possible issue is "new similar task", not sure if I want to bring that into ApplicationEditor or not yet. I may just sort of hack around it for v0, although having "new similar object" in every application doesn't necessarily seem terrible to me, just also not clearly useful/desirable in many apps.)

webroot/rsrc/js/application/transactions/behavior-comment-actions.js
77–82

aww yeahhh

Yeah, maybe for mobile... or everywhere... just update the select to be "Remove Projects", then we don't need all that code for adding icons?

Two possible issues with that:

  • That might get hairy if we stick with the direct-status-actions in Differential ("Remove Accept Revision" next to "Request Changes"?). Not totally sure what that will look like yet.
  • This interface will also be used in the EditEngine version of "Batch Editor", but it will have a "Remove Projects" action there, so we'd end up with "Remove 'Remove Projects' Action'" or similar.

I think we can get it working OK on mobile in the relatively short term through brute force and then refine this later once we have a clearer idea about other use cases.

The "use a dropdown to pick actions" UI feels a little odd to me, but I think I'll get used to it. It does seem a bit cleaner than the sort of Herald approach (where you select each action from a separate dropdown) in this situation (although I like the Herald approach for Herald).

This revision was automatically updated to reflect the committed changes.