Page MenuHomePhabricator

Warn and prevent 1-up/2-up switch in Differential if the user is editing an inline
ClosedPublic

Authored by epriestley on Nov 15 2017, 5:28 PM.
Tags
None
Referenced Files
F19791916: D18774.diff
Feb 26 2026, 1:47 AM
F19531648: D18774.diff
Jan 20 2026, 6:10 AM
F19528002: D18774.diff
Jan 19 2026, 6:12 AM
F19103690: D18774.id45050.diff
Dec 5 2025, 11:33 AM
F19100275: D18774.diff
Dec 4 2025, 11:01 PM
F17830359: D18774.id45050.diff
Jul 26 2025, 11:01 AM
F17817377: D18774.id.diff
Jul 25 2025, 11:30 PM
F17801413: D18774.diff
Jul 25 2025, 7:46 AM
Subscribers
None

Details

Summary

See PHI180. Currently, if you begin creating or editing an inline and then swap display modes (for example, with "View Unified"), your edit is lost.

Persisting the editor state is complicated and this is very rare, so just prevent the action and warn the user instead.

Also make the warning persist for a little longer since a few of the messages, including this one, take a couple seconds to read now.

Test Plan
  • Edited a comment, tried to swap display modes, got a warning.
  • Swapped display modes normally with no comment being edited.

Diff Detail

Repository
rP Phabricator
Branch
change1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 18832
Build 25384: Run Core Tests
Build 25383: arc lint + arc unit