fix(ui): improve message scroll behavior to ensure bottom content visibility #357
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ibility - Implement dual-phase scrolling: virtualizer positioning + native scroll to bottom - Reduce excessive bottom padding from pb-40 to pb-20 for better viewport usage - Unify scroll behavior across auto-scroll, history loading, and manual scroll - Fix issue where streamed content bottom couldn't reach viewport
- Add IME composition state tracking to prevent premature submission during input composition - Handle onCompositionStart/End events in text inputs and textareas - Replace onKeyPress with onKeyDown to better handle IME interactions - Add proper IME detection in FloatingPromptInput for enhanced input handling - Fix issues with Enter key triggering actions during IME composition in: - AgentExecution task input - ClaudeCodeSession fork dialog - TimelineNavigator checkpoint creation - WebviewPreview URL input - FloatingPromptInput main textarea This ensures proper input behavior for users of CJK input methods and other IME systems.
kmk142789
approved these changes
Sep 22, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Fixes issue where streamed content bottom couldn't reach viewport during message streaming.
Changes
Technical Details
The fix implements a two-step scrolling approach:
rowVirtualizer.scrollToIndex()to quickly position near the targetscrollElement.scrollTo()to ensure absolute bottom is reachedThis maintains virtualization performance while guaranteeing content visibility.