Page MenuHomePhabricator
Differential D21219

When cancelling an unsaved editing inline after a reload, don't cancel into an empty state
ClosedPublic
Actions

Authored by epriestley on May 4 2020, 9:01 PM.
Tags
None
Referenced Files
F15464789: D21219.id50535.diff
Wed, Apr 2, 12:47 PM
F15457724: D21219.diff
Sun, Mar 30, 6:10 PM
F15457389: D21219.diff
Sun, Mar 30, 4:03 PM
F15441886: D21219.id50535.diff
Wed, Mar 26, 8:31 PM
F15430774: D21219.id50532.diff
Mon, Mar 24, 8:52 AM
F15424245: D21219.id50532.diff
Sat, Mar 22, 9:06 PM
F15423017: D21219.id.diff
Sat, Mar 22, 12:08 PM
F15415786: D21219.id50535.diff
Thu, Mar 20, 7:52 AM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13513: Make schema changes to inlines to prepare for new inline features
Commits
rP07e160bde1e8: When cancelling an unsaved editing inline after a reload, don't cancel into an…
Summary

Ref T13513. Overloading "original text" to get "edit-on-load" comments into the right state has some undesirable side effects.

Instead, provide the text when the editor opens. This fixes a cancel interaction.

Test Plan
  • Create an inline, type text, don't save.
  • Reload page.
  • Cancel.
  • Before: cancelled into empty state.
  • After: cancelled into deleted+undo state.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.May 4 2020, 9:01 PM
Harbormaster completed remote builds in B24349: Diff 50532.May 4 2020, 9:02 PM
epriestley requested review of this revision.May 4 2020, 9:02 PM
This revision was not accepted when it landed; it landed in state Needs Review.May 4 2020, 10:20 PM
Closed by commit rP07e160bde1e8: When cancelling an unsaved editing inline after a reload, don't cancel into an… (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rP07e160bde1e8: When cancelling an unsaved editing inline after a reload, don't cancel into an….

Revision Contents
Changeset List

PathSize
resources/
celerity/
src/
infrastructure/
diff/
8 lines
view/
2 lines
4 lines
webroot/
rsrc/
js/
application/
diff/
2 lines

Diff 50535

View Options

resources/celerity/map.php

Loading...
View Options

src/infrastructure/diff/PhabricatorInlineCommentController.php

Loading...
View Options

src/infrastructure/diff/view/PHUIDiffInlineCommentEditView.php

Loading...
View Options

src/infrastructure/diff/view/PHUIDiffInlineCommentView.php

Loading...
View Options

webroot/rsrc/js/application/diff/DiffInline.js

Loading...
Phabricator · Built by Phacility