1
90%
master: 90%

Ran 11 May 2026 05:14PM UTC

Files 69

Run time 3s

Badge

Embed ▾

Committed 11 May 2026 05:13PM UTC coverage: 89.764% (+0.06%) from 89.703%

Job # 25685448594.1

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

9752 of 10864 relevant lines covered (89.76%)

3193.43 hits per line

umputun / revdiff / 25685448594 / 1
90%
master: 90%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 25685448594.1

umputun / revdiff / 25685448594 / 1 90% master: 90%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 25685448594.1

umputun / revdiff / 25685448594 / 1
90%
master: 90%

README BADGES
x