umputun / revdiff / 25687579295

90%

push

github

fix(mouse): tree/TOC wheel scrolls one entry per notch (not three) (#181)

* fix(mouse): tree/TOC wheel = direct cursor nav, one entry per notch

Previously the hitTree branch passed wheelStep (3) to MotionPageDown/Up, so each wheel notch skipped 3 entries in the file tree and 3 sections in the TOC. Shift+wheel further amplified to half-page.

That's the diff-pane mental model bleeding into the sidebar. The sidebar is small and cheap; users expect j/k-style single-step navigation per wheel notch — same as every IDE/editor file tree.

Switched hitTree to call Move(MotionDown) / Move(MotionUp) (no count), ignoring the delta magnitude entirely. wheelStepFor no longer needs the hitZone param (only diff-pane consults it), so dropped that and simplified the function. Deleted the now-obsolete tree-half-page test and rewrote the TOC wheel test to use two notches (ParseTOC prepends a synthetic filename entry at lineIdx=0, so one notch from the start lands on a heading that shares lineIdx=0 — two notches lands on a heading with a distinct lineIdx).

* fix(review-loop): iteration 1 addressed 11 findings

- [major] README.md:710-711 — "Three lines per notch" and "Shift+scroll half-page" were claimed across all panes; now correctly scoped (tree/TOC = single-step regardless of shift, diff = wheelStep / half-page)
- [major] .claude-plugin/skills/revdiff/references/usage.md — same correction, kept byte-identical with README mouse section
- [major] plugins/codex/skills/revdiff/references/usage.md — same correction
- [major] site/docs.html:561-562 — same correction (must mirror README)
- [minor] CLAUDE.md:98 — "across panes" rewritten to scope wheelStep to diff + overlay only; tree/TOC ignores the constant
- [test-gap] mouse_test.go WheelInTreeMovesTreeCursor — broken up into 6 subtests asserting exact one-entry advance, wheel-up direction, shift-still-single-step, top/bottom clamp, and the negative invariant that tree wheel must not touch diff wheelState
- [test-gap] ... (continued)

10 of 10 new or added lines in 1 file covered. (100.0%)

9753 of 10862 relevant lines covered (89.79%)

3194.99 hits per line