umputun / revdiff / 25206350155

90%

master: 90%

Build Type

Pull #162

github

Committed by umputun

Commit Message

fix(ui): land cursor on annotation comment row when jumping via @ or }/{

positionOnAnnotation set diffCursor to the annotated diff line index but
left m.annot.cursorOnAnnotation=false. The cursor render path checks both:
the cursor highlight only moves to the annotation comment sub-row when
cursorOnAnnotation is true. As a result, both @ popup selection and the
new }/{ navigation visually placed the cursor on the diff line above the
comment, one row off the actual annotation.

Set cursorOnAnnotation=true after positioning, matching the invariant that
j/k navigation already maintains when stepping onto an annotated line.
File-level annotations (Line=0) are unaffected — diffCursor=-1 already
represents the annotation row directly, so the flag stays false there.

Skips the flag when the line is collapsed-hidden or a delete-only
placeholder, where the annotation comment row does not render.

Pull Request Pull Request #162: feat: navigate annotations across files with }/{ keys

Coverage Stats

109 of 110 new or added lines in 4 files covered. (99.09%)

1 existing line in 1 file now uncovered.

9447 of 10533 relevant lines covered (89.69%)

514.97 hits per line