coracle/flotilla
1
6
Fork 24
Code Issues 42 Pull Requests 2 Actions Packages Projects 5 Releases Wiki Activity

fix(chat): suppress programmatic scroll while user is scrolling (#132) #216

Merged
hodlbod merged 1 commits from Khushvendra/flotilla:fix/scroll-jitter-132 into dev 2026-04-16 23:20:17 +00:00
Conversation 1 Commits 1 Files Changed 2
+18 -2
Khushvendra commented 2026-04-16 23:17:44 +00:00
Contributor
Copy Link
Copy Source

Closes #132.

Summary
Scrolling in reversed chat layouts could jitter and jump to extreme positions during mouse wheel and trackpad use. The core issue was programmatic scroll management running while the user was actively scrolling and while new event batches were re-rendering.

Root Cause

  • The elements-driven effect re-ran scroll management on event updates.
  • Feed updates during active scroll caused frequent DOM/layout changes.
  • There was no active-user-scroll guard to pause programmatic adjustments.

Changes made

  • Added an isUserScrolling state flag in both chat views.
  • Added a 150ms debounced clear using throttle-debounce to detect scroll end.
  • In onScroll, only user-initiated scrolls set isUserScrolling.
  • Guarded the elements effect so manageScrollPosition does not run while isUserScrolling is true.
  • Kept existing programmatic scroll behavior intact, including deep-link positioning via at and normal scroll-to-bottom/new-message actions.
  • No changes were made to feed loading logic in requests.

Validation

  • ESLint passed for both touched files.
  • Full repository typecheck currently fails due unrelated pre-existing errors in poll-related imports/types and capacitor share typing, not introduced by this PR.
Closes #132. **Summary** Scrolling in reversed chat layouts could jitter and jump to extreme positions during mouse wheel and trackpad use. The core issue was programmatic scroll management running while the user was actively scrolling and while new event batches were re-rendering. **Root Cause** - The elements-driven effect re-ran scroll management on event updates. - Feed updates during active scroll caused frequent DOM/layout changes. - There was no active-user-scroll guard to pause programmatic adjustments. **Changes made** - Added an `isUserScrolling` state flag in both chat views. - Added a 150ms debounced clear using throttle-debounce to detect scroll end. - In `onScroll`, only user-initiated scrolls set `isUserScrolling`. - Guarded the elements effect so `manageScrollPosition` does not run while `isUserScrolling` is true. - Kept existing programmatic scroll behavior intact, including deep-link positioning via at and normal scroll-to-bottom/new-message actions. - No changes were made to feed loading logic in requests. **Validation** - ESLint passed for both touched files. - Full repository typecheck currently fails due unrelated pre-existing errors in poll-related imports/types and capacitor share typing, not introduced by this PR.
Khushvendra added 1 commit 2026-04-16 23:17:45 +00:00
fix(chat): suppress programmatic scroll while user is scrolling (#132) b426fde43e
hodlbod commented 2026-04-16 23:20:13 +00:00
Owner
Copy Link
Copy Source

Nice, this is an elegant solution.

Nice, this is an elegant solution.
hodlbod merged commit 4a967de184 into dev 2026-04-16 23:20:17 +00:00
hodlbod deleted branch fix/scroll-jitter-132 2026-04-16 23:20:17 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
design
For issues related to design work. Development should be done in another issue and refer back to this one.
 dev
Issues related to development work
 easy
Issues that can be executed on by new contributors
 idea
For issues that aren't yet actionable
 priority
For issues related to high priority features and bugs
 usability
Issues that have to be validated with real live human beings.
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
hodlbod (hodlbod) mplorentz (Matt Lorentz) sakshamjain (Saksham Jain) sondreb triesap userAdityaa (Aditya Chaudhary)
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: coracle/flotilla#216
Delete Branch "Khushvendra/flotilla:fix/scroll-jitter-132"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?