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
F18694353: D17905.diff
Sat, Sep 27, 3:31 AM
F18670951: D17905.id43067.diff
Thu, Sep 25, 2:14 AM
F18508806: D17905.id.diff
Sep 5 2025, 3:02 AM
F18501054: D17905.diff
Sep 4 2025, 9:31 PM
F18416978: D17905.id.diff
Aug 30 2025, 11:15 AM
F18396094: D17905.diff
Aug 29 2025, 11:49 AM
F18103751: D17905.id43067.diff
Aug 10 2025, 7:37 AM
F18090763: D17905.id43067.diff
Aug 6 2025, 5:24 PM
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