Page MenuHomePhabricator

Make new DiffInline code handle most "delete" operations
ClosedPublic

Authored by epriestley on May 15 2017, 10:10 PM.
Tags
None
Referenced Files
F15536565: D17890.id43052.diff
Thu, Apr 24, 10:33 AM
F15517326: D17890.id.diff
Sat, Apr 19, 2:10 AM
F15513290: D17890.id43030.diff
Thu, Apr 17, 5:52 PM
F15481775: D17890.id43030.diff
Tue, Apr 8, 10:44 PM
F15480142: D17890.id43052.diff
Tue, Apr 8, 12:07 PM
F15475758: D17890.id.diff
Sun, Apr 6, 11:25 PM
F15475231: D17890.diff
Sun, Apr 6, 5:45 PM
F15472622: D17890.diff
Sat, Apr 5, 3:03 PM
Subscribers
None

Details

Summary

Ref T12616. This moves the delete actions to the new, more stateful way of doing things.

These are a little tricky because you can click "Delete" on an inline, but you can also click "Delete" from the preview area at the bottom of the page. If you do, the inline you are deleting may or may not be present on the page.

This has a few bugs -- notably, deleting from the preview without interacting with the on-page inline first won't actually delete the on-page inline yet -- but nothing too serious.

Test Plan

Deleted inlines, undid deletion, deleted from preview.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable