Skip to content

Roaming cursor for selecting inside blocks - #75

Merged
backnotprop merged 6 commits into
plannotator:mainfrom
lawrencegripper:list-items-and-roaming-cursor
Sep 20, 2026
Merged

backnotprop merged 6 commits into
plannotator:mainfrom
lawrencegripper:list-items-and-roaming-cursor

Conversation

@lawrencegripper

Copy link
Copy Markdown
Contributor

Love the project, thanks for working on it, been massively improvement in my loop using this with the herdr plugin.

PR addresses one nit where I want to comment on a single line from a block.

This PR adds that with: o starts movable cursor, c to start copy, v to stop then c to comment

CleanShot.2026-09-18.at.13.16.16.mp4

A list was one block, so j/k stepped over a whole findings list and c
commented on all of it. Items open a block each; nested content stays
with its item.
Block mode had no row movement, so a keyboard selection always anchored
at the block's first row. o enters roaming: the visual motions move the
cursor with nothing selected, v anchors there, Esc or a block key
returns. Motions are shared with visual mode.
Roaming reaches any line of a list, so splitting it into item blocks
bought little and changed rendering (a block gap between bullets) and
the block model for everyone. Decision 15 records why.
The first version used three one-line items, so j moving a row and j
moving a block were indistinguishable.
Closer to vim: i steps into the block the way insert mode steps into a
line, and o carries unrelated meanings (open a line, swap visual ends).
@jrobchin

Copy link
Copy Markdown

Nice I just create an issue for this 😄 #76

h, l and the arrow keys moved the cursor in block mode with nothing drawn, so v
anchored at a column the user never saw. They now enter roaming first, which
draws the cursor, then move.

refs plannotator#76
@backnotprop
backnotprop added this pull request to the merge queue Sep 20, 2026
Merged via the queue into plannotator:main with commit 44ee9e5 Sep 20, 2026
2 checks passed
@backnotprop

Copy link
Copy Markdown
Contributor

Merged and released in v0.9.0. Thanks for this; the decision note was exactly the kind of write-up we like. I added one commit on top so block-mode h/l and the arrows enter roaming too, which closed #76 alongside it.

@jrobchin

Copy link
Copy Markdown

Huge QoL fix thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants