Page MenuHomePhabricator

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

Authored by epriestley on May 4 2020, 9:01 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 1 2024, 1:22 AM
Unknown Object (File)
Dec 30 2023, 5:36 PM
Unknown Object (File)
Dec 27 2023, 1:23 PM
Unknown Object (File)
Dec 26 2023, 7:06 PM
Unknown Object (File)
Dec 23 2023, 3:00 AM
Unknown Object (File)
Dec 22 2023, 2:57 AM
Unknown Object (File)
Dec 19 2023, 4:30 PM
Unknown Object (File)
Dec 18 2023, 12:54 PM
Subscribers
None

Details

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

This revision was not accepted when it landed; it landed in state Needs Review.May 4 2020, 10:20 PM
This revision was automatically updated to reflect the committed changes.