Page MenuHomePhabricator

Refine unusual inline comment client interactions
ClosedPublic

Authored by epriestley on May 4 2020, 4:33 PM.
Tags
None
Referenced Files
F13195120: D21212.diff
Sun, May 12, 10:08 PM
F13177898: D21212.diff
Wed, May 8, 8:02 PM
Unknown Object (File)
Tue, May 7, 8:04 AM
Unknown Object (File)
Sun, May 5, 1:48 PM
Unknown Object (File)
Thu, May 2, 12:06 PM
Unknown Object (File)
Wed, May 1, 7:34 AM
Unknown Object (File)
Thu, Apr 25, 12:57 AM
Unknown Object (File)
Sat, Apr 20, 1:21 AM
Subscribers
None

Details

Summary

Ref T13513. Refine some inline behaviors, see test plan.

Test Plan
  • Edit a comment ("A"), type text ("AB"), cancel, edit.
    • Old behavior: edit and undo states (wrong, and undo does not function).
    • New behavior: edit state only.
  • Edit a comment ("A"), type text ("AB"), cancel. Undo ("AB"), cancel. Edit.
    • Old behavior: "AB" (wrong: you never submitted this text).
    • New behavior: "A".
  • Create a comment, type text, cancel.
    • Old behavior: counter appears in filetree (wrong, comment is undo-able but should not be counted).
    • New behavior: no counter.
  • Cancel editing an empty comment with no text.
    • Old behavior: Something buggy -- undo, I think?
    • New behavior: it just vanishes (correct behavior).

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, 8:15 PM
This revision was automatically updated to reflect the committed changes.