forked from huggingface/chat-ui
-
Notifications
You must be signed in to change notification settings - Fork 0
Fix auto-scroll behavior for chat messages #15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Closed
gary149
wants to merge
1,679
commits into
main
from
claude/fix-auto-scroll-messages-019JCykHCXX1FtuH49NBsQiF
Closed
Fix auto-scroll behavior for chat messages #15
gary149
wants to merge
1,679
commits into
main
from
claude/fix-auto-scroll-messages-019JCykHCXX1FtuH49NBsQiF
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
Owner
gary149
commented
Nov 30, 2025
- Add ResizeObserver to detect content height changes (expanding thinking blocks, streaming content)
- Make detach detection less aggressive with proper thresholds
- Track programmatic vs user scrolls to avoid false detachment
- Use requestAnimationFrame for smoother scroll updates
- Update ScrollToBottomBtn and ScrollToPreviousBtn with consistent thresholds
Fix touchscreen detection
…ace#1825) refactor: improve rendering perf. on low-end hardware - Reintroduced MarkdownWorker for asynchronous processing of markdown content. - Debounce message content updates dynamically based on render time
Bumps [undici](https://github.com/nodejs/undici) from 5.28.5 to 5.29.0. - [Release notes](https://github.com/nodejs/undici/releases) - [Commits](nodejs/undici@v5.28.5...v5.29.0) --- updated-dependencies: - dependency-name: undici dependency-version: 5.29.0 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…1829) allows you to skip building llama.cpp if you know you are not going to use it
Signed-off-by: Adrien <adrien@huggingface.co>
Signed-off-by: Adrien <adrien@huggingface.co>
…uggingface#1834) fix: remove quotes from search query to prevent exact match search when running locally using serper Co-authored-by: Nathan Sarrazin <sarrazin.nathan@gmail.com>
…uggingface#1692) * feat(test): add support for testing svelte components in isolation * refactor: use `vitest-browser-svelte` for tests, fix markdownRenderer tests * test: remove todos on tests that pass * fix: deps & tests
…versation history (huggingface#1823) * UI for crtl/command + k for conversation search * UI for ctrl/command+k * Added fetch conversations and filter conversations functions * WIP - finish with the first draft of popup-search * WIP - Finish the first version pop up search * WIP - Finish the first version pop up search * Remove unnecessary codes * Fixed formatting errors * Fixed formatting errors * refactor: search functionality * fix: revert those changes * fix: lint * chores: clean up unused comment --------- Co-authored-by: kenneth.ramos95 <kenneth.ramos95@gmail.com> Co-authored-by: Nathan Sarrazin <sarrazin.nathan@gmail.com>
…face#1837) * feat(ci): add a health check after building the docker image * fix: change group for running health check * fix: newline in command * fix: docker container image name * fix: its goign to work a lot better if you run the container first * temp: try to break ci * temp: try to break ci pt 2 * temp: revert broken ci, healthcheck works yay!
…ME (huggingface#1835) feat: Mention incoming Model Context Protocol (MCP) support in README Adds a new section to the README.md under "Custom models" detailing the planned support for the Model Context Protocol (MCP). This section briefly explains MCP and its potential benefits for Chat UI, such as enhanced flexibility and extensibility in how models receive and understand context. It also includes a call to action for contributors to help with the design, implementation, and integration of MCP, directing them to the issue tracker for relevant tasks. Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
* Update overview.md * Update web-search.md
…ggingface#1840) Ensures textarea resizes properly before message submission with Enter key
* Added infinite scrolling to search feature. * Added code to highlight the search keyword in search description * feat: improve description layout * feat: improve search context finding * fix: remove all debug bits --------- Co-authored-by: Nathan Sarrazin <sarrazin.nathan@gmail.com>
…gingface#1838) * feat(huggingchat): model update 27-05-25 * feat: update prompt examples in config * feat: revert model config changes
…wrapper (huggingface#1845) fix: remove {#key} wrapper to prevent unnecessary component recreation (MardownRenderer) - Remove {#key message.content} wrapper from ChatMessage component - Allow MarkdownRenderer to handle content updates via reactive props - Eliminate component destruction/recreation on every content change - Improve streaming performance by reusing component instances
…nd fallback models for enhanced functionality
… clarity and consistency
…upport; update README for clarity on OpenAI compatibility
…update privacy policy reference and improve Markdown rendering
… icon sizes and paths
…gle image element
…eNav and ChatWindow components
- Add ResizeObserver to detect content height changes (expanding thinking blocks, streaming content) - Make detach detection less aggressive with proper thresholds - Track programmatic vs user scrolls to avoid false detachment - Use requestAnimationFrame for smoother scroll updates - Update ScrollToBottomBtn and ScrollToPreviousBtn with consistent thresholds
- Reduce SCROLL_UP_THRESHOLD from 30px to 5px so users can detach easily - Add 150ms cooldown after user scroll to prevent auto-scroll from fighting - Track lastUserScrollTime to detect active user scrolling
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.