umputun / revdiff / 25685448594

90%

master: 90%

Build Type

Pull #180

github

Committed by umputun

Commit Message

fix(wheel): skip redundant render in flushWheelPending when cursor stays in view

Copilot review on #180 caught that flushWheelPending unconditionally re-renders even when pinDiffCursorTo returns false (cursor stayed visible through the burst). In that case the existing viewport content already has the correct cursor highlight, so SetContent(renderDiff()) is wasted work — and on the handleKey path it double-renders because the key handler will also render after its action runs.

Gate syncTOCActiveSection + SetContent on the pin's bool return. Always clear renderPending and tickInFlight either way. Also dropped a redundant tickInFlight=false in handleWheelDebounce — flushWheelPending already clears it.

New test TestModel_HandleMouse_WheelDebounceMsg_SkipsRenderWhenCursorStaysInView locks in the contract: a flush with cursor still in view must not change viewport.View().

Pull Request Pull Request #180: fix: coalesce diff-pane wheel events to unblock reverse scroll (#179)

Coverage Stats

61 of 62 new or added lines in 3 files covered. (98.39%)

1 existing line in 1 file now uncovered.

9752 of 10864 relevant lines covered (89.76%)

3193.43 hits per line