Page MenuHomePhabricator

Unify EditEngine preview behavior; prepare for saving complex drafts
ClosedPublic

Authored by epriestley on Dec 4 2015, 10:13 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Feb 19, 5:30 AM
Unknown Object (File)
Wed, Feb 19, 5:29 AM
Unknown Object (File)
Wed, Feb 19, 5:29 AM
Unknown Object (File)
Tue, Feb 11, 2:35 PM
Unknown Object (File)
Sat, Feb 8, 11:59 PM
Unknown Object (File)
Jan 25 2025, 1:17 PM
Unknown Object (File)
Jan 24 2025, 10:16 AM
Unknown Object (File)
Jan 21 2025, 9:26 AM
Subscribers
None

Details

Summary

Ref T9132. We currently have an old preview/draft behavior and a new actions behavior.

Let the actions behavior do drafts/previews too, so we can eventually throw away the old thing.

This is pretty much just copying the old behavior into the new one, but with a few tweaks. The major change is that we submit all the stacked actions behavior now, so the preview reflects everything the change will do (and, soon, we can save it in the draft in a consistent way).

Also includes one hack-fix that I'll clean up at some point.

Test Plan

Added a bunch of stacked actions and observed meaningful previews.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Unify EditEngine preview behavior; prepare for saving complex drafts.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Dec 4 2015, 10:46 PM
This revision was automatically updated to reflect the committed changes.