Skip to content

Comments

fix: Scroll improvements and visibility logging#465

Merged
m13v merged 2 commits intomainfrom
pr/scroll-fixes-20260127-144039
Jan 27, 2026
Merged

fix: Scroll improvements and visibility logging#465
m13v merged 2 commits intomainfrom
pr/scroll-fixes-20260127-144039

Conversation

@m13v
Copy link
Contributor

@m13v m13v commented Jan 27, 2026

Summary

  • Add oscillation and stale bounds detection to scroll_into_view to prevent infinite loops
  • Stops scrolling if direction flips 3+ times or bounds unchanged for 4 steps
  • Improve is_visible debug logging with element names and bounds

Test plan

  • Test scroll_into_view with elements that don't move when scrolled
  • Verify no infinite scroll loops occur

Generated with Claude Code

m13v and others added 2 commits January 27, 2026 14:39
Stops scrolling if direction flips 3+ times or bounds unchanged for 4 steps to prevent infinite loops.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@m13v m13v merged commit bd523a3 into main Jan 27, 2026
2 checks passed
@m13v m13v deleted the pr/scroll-fixes-20260127-144039 branch January 27, 2026 22:41
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