Page MenuHomePhabricator

When draft inline comments are submitted, disengage the editor
ClosedPublic

Authored by epriestley on Apr 29 2020, 7:00 PM.
Tags
None
Referenced Files
F13058877: D21188.diff
Fri, Apr 19, 3:20 PM
Unknown Object (File)
Mon, Apr 1, 7:47 PM
Unknown Object (File)
Mon, Apr 1, 2:10 AM
Unknown Object (File)
Sun, Mar 31, 6:32 AM
Unknown Object (File)
Fri, Mar 29, 9:11 PM
Unknown Object (File)
Fri, Mar 29, 6:27 PM
Unknown Object (File)
Feb 22 2024, 3:17 PM
Unknown Object (File)
Feb 8 2024, 11:08 AM
Subscribers
None

Details

Summary

Ref T13513. If you submit top-level comments while an inline comment editor is open, kick the comment out of the editing state.

(An improvement to this behavior would be to warn the user that we're going to do this first, but this is currently less straightforward.)

Test Plan
  • Clicked a line number to create an inline.
  • Type text, save, click edit.
  • (Optional: reload page.)
  • Save changes overall using the form at the bottom of the page.
  • Outcome: published inline is no longer in an "editing" state.

Weirdness:

  • If you click a line number (and, optionally, type text), then submit without using "Save", the server-side version of the inline has no content.
    • This gives you a no-effect warning. Instead, these inlines should probably just be marked as deleted somewhere in the pipeline.
  • This saves the last "Saved" copy of the inline. That's (probably?) desired, but somewhat destructive without a warning.

Diff Detail

Repository
rP Phabricator
Branch
inline4
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 24275
Build 33439: Run Core Tests
Build 33438: arc lint + arc unit