Page MenuHomePhabricator

Retain keyboard cursor state across more inline edit operations in Differential
ClosedPublic

Authored by epriestley on May 16 2017, 2:31 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 11, 10:12 AM
Unknown Object (File)
Fri, Apr 5, 1:41 AM
Unknown Object (File)
Sun, Mar 31, 10:17 PM
Unknown Object (File)
Sun, Mar 31, 6:55 AM
Unknown Object (File)
Mar 21 2024, 12:42 AM
Unknown Object (File)
Jan 17 2024, 7:04 PM
Unknown Object (File)
Jan 14 2024, 5:48 PM
Unknown Object (File)
Jan 12 2024, 12:38 AM
Subscribers
None

Details

Summary

Ref T12634. Fixes T8131. Currently, most edit operations (edit, reply, collapse, mark done) lose the keyboard cursor state.

Instead, bind the state more tighlty to the inline object itself (instead of the rows which happen to be in the document), and then do a bit of recalculation to try to keep it selected across edits.

Test Plan
  • Used "n" to select an inline.
  • Clicked "Done" checkbox.
  • Pressed "n".
  • Went to the next inline (previously: lost position in document).
  • Behavior is also better for: edit, reply, collapse/expand.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable