Skip to content

VirtualList: fire onScroll on scroll changes#21

Merged
RtlZeroMemory merged 2 commits into
mainfrom
feat/virtuallist-onscroll
Feb 11, 2026
Merged

VirtualList: fire onScroll on scroll changes#21
RtlZeroMemory merged 2 commits into
mainfrom
feat/virtuallist-onscroll

Conversation

@RtlZeroMemory

Copy link
Copy Markdown
Owner

Fix VirtualList onScroll wiring:

  • Invoke onScroll for key-driven and wheel scroll changes
  • Compute the visible range via computeVisibleRange (matches render-time logic)
  • Add WidgetRenderer integration test covering wheel + key-driven scroll
  • Document onScroll range semantics

Checks:

  • npm run lint
  • npm run typecheck -- --force
  • npm test (874)
  • npm run test:e2e skipped on non-Linux by design

@RtlZeroMemory
RtlZeroMemory force-pushed the feat/virtuallist-onscroll branch from 67f496f to ae2626c Compare February 11, 2026 07:39
Invoke onScroll for key-driven and wheel scroll changes and compute the visible range via the same computeVisibleRange logic used for rendering.
Add an integration test for wheel + key-driven scroll callbacks and document the onScroll range semantics.
@RtlZeroMemory
RtlZeroMemory force-pushed the feat/virtuallist-onscroll branch from ae2626c to c347b7f Compare February 11, 2026 07:43
@RtlZeroMemory
RtlZeroMemory merged commit fbe5d0e into main Feb 11, 2026
14 checks passed
@RtlZeroMemory
RtlZeroMemory deleted the feat/virtuallist-onscroll branch February 11, 2026 07:47
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.

1 participant