Skip to content

Release web scroll follow on input intent - #18

Merged
MinionEnjoyer merged 1 commit into
mainfrom
codex/fix-web-scroll-intent
Aug 15, 2026
Merged

Release web scroll follow on input intent#18
MinionEnjoyer merged 1 commit into
mainfrom
codex/fix-web-scroll-intent

Conversation

@MinionEnjoyer

Copy link
Copy Markdown
Owner

What changed

  • Release automatic bottom-follow during upward wheel or trackpad input, before the browser dispatches scroll.
  • Apply the same early intent handling to touch scrolling and Page Up, Home, and Arrow Up navigation.
  • Cancel pending restoration animation frames when the reader expresses history-navigation intent.
  • Strengthen the browser regression so the wheel event occurs before the scroll position changes.

Root cause

The first fix correctly reacted to upward scroll, but browsers can materialize content-visibility rows and invoke ResizeObserver after wheel input and before the scroll event. During that interval, bottom-follow remained enabled and the observer could snap the viewport back to the newest edge.

Validation

  • Negative control reproduced the premature snap from scrollTop 760 back to 1000.
  • 75/75 web unit tests passed.
  • TypeScript and Vite production build passed.
  • Complete Playwright harness passed: 12 passed, 8 intentional project-specific skips.
  • Independent focused verification passed: 10 scroll tests and the desktop Chromium wheel-before-scroll regression.

This changes desktop/web and responsive web behavior only; native phone application code is untouched.

@MinionEnjoyer
MinionEnjoyer marked this pull request as ready for review August 15, 2026 02:45
@MinionEnjoyer
MinionEnjoyer merged commit eb743bd into main Aug 15, 2026
5 checks passed
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