Change Details

The display layer for inline comments hasn't been updated in a long time is currently under-engineered. We have a large number of glitchy interactions because the various components are too loosely organized. For example, the keyboard selection reticle and inline edit workflow basically break every time they interact between them because they aren't really aware of one another. I plan to rework the display layer to generally follow the patterns present in the Workboards code. (Conpherence will also get an update in this vein in T12562.)

The display layer for inline comments hasn't been updated in a long time is currently under-engineered. We have a large number of glitchy interactions because the various components are too loosely organized. For example, the keyboard selection reticle and inline edit workflow basically break every time they interact between them because they aren't really aware of one another. I plan to rework the display layer to generally follow the patterns present in the Workboards code. (Conpherence will also get an update in this vein in T12562.) ---- **TODOS** [ ] Restore "h" to collapse or expand files. [ ] Restore "r" to reply to inlines. [ ] Restore "e" to edit inlines. [ ] Determine if we need to cache cursor state DOM information for keyboard navigation ("n", "p", etc) for performance.

The display layer for inline comments hasn't been updated in a long time is currently under-engineered. We have a large number of glitchy interactions because the various components are too loosely organized. For example, the keyboard selection reticle and inline edit workflow basically break every time they interact between them because they aren't really aware of one another. I plan to rework the display layer to generally follow the patterns present in the Workboards code. (Conpherence will also get an update in this vein in T12562.) ---- **TODOS** [ ] Restore "h" to collapse or expand files. [ ] Restore "r" to reply to inlines. [ ] Restore "e" to edit inlines. [ ] Determine if we need to cache cursor state DOM information for keyboard navigation ("n", "p", etc) for performance.