Skip to content

fix(web): allow reasoning scroll chaining - #1264

Merged
tiann merged 1 commit into
tiann:mainfrom
teekalpha:fix/reasoning-scroll-chaining
Jul 31, 2026
Merged

fix(web): allow reasoning scroll chaining#1264
tiann merged 1 commit into
tiann:mainfrom
teekalpha:fix/reasoning-scroll-chaining

Conversation

@teekalpha

Copy link
Copy Markdown
Contributor

Summary

Testing

  • bun typecheck
  • bun run test:web

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Findings

  • No findings.

Questions

  • None.

Summary

  • Review mode: initial. No issues found in the added/modified line. Removing overscroll-contain restores default scroll chaining from the nested reasoning panel to the chat viewport; the outer .app-scroll-y still contains page-level overscroll via web/src/index.css:313.
  • Residual risk: browser-specific wheel, touchpad, and touch boundary behavior is not covered by the repository’s unit tests.

Testing

  • Not run (automation). GitHub test check was pending at review time; the PR reports bun typecheck and bun run test:web.

HAPI Bot

@tiann
tiann merged commit 212e58c into tiann:main Jul 31, 2026
2 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.

2 participants