Skip to content

webui: Fix Chat Screen Form box disappearing + autoscroll issues on WebKit#22977

Merged
allozaur merged 4 commits into
ggml-org:masterfrom
allozaur:21276-webui-chat-screen-form-box-disappearing
May 12, 2026
Merged

webui: Fix Chat Screen Form box disappearing + autoscroll issues on WebKit#22977
allozaur merged 4 commits into
ggml-org:masterfrom
allozaur:21276-webui-chat-screen-form-box-disappearing

Conversation

@allozaur

@allozaur allozaur commented May 12, 2026

Copy link
Copy Markdown
Contributor

Overview

Close #21276

Fixes two WebKit-specific bugs in the chat UI:

  1. Chat form vanishes — Switched scroll container from flex-col-reverse to flex-col.

  2. Autoscroll glitches — also caused by flex-col-reverse

Requirements

@allozaur allozaur marked this pull request as ready for review May 12, 2026 15:38
@allozaur allozaur requested a review from a team as a code owner May 12, 2026 15:38

Copilot AI 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.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Fixes WebKit-specific chat UI issues by changing the chat scroll layout and improving auto-scroll and transition behavior during navigation/conversation changes.

Changes:

  • Switch chat scroll container from flex-col-reverse to flex-col and adjust auto-scroll logic accordingly.
  • Add a “messages ready” callback to trigger an immediate scroll-to-bottom on navigation.
  • Introduce fade-in/out visibility transitions when switching conversations and during navigation.

Reviewed changes

Copilot reviewed 3 out of 5 changed files in this pull request and generated 4 comments.

File Description
tools/server/webui/src/lib/hooks/use-auto-scroll.svelte.ts Removes column-reverse support and simplifies bottom detection/scroll-to-bottom behavior.
tools/server/webui/src/lib/components/app/chat/ChatScreen/ChatScreen.svelte Updates scroll container direction and triggers immediate scroll-to-bottom when messages are ready.
tools/server/webui/src/lib/components/app/chat/ChatMessages/ChatMessages.svelte Adds navigation/conversation transition handling and emits onMessagesReady callback.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread tools/server/webui/src/lib/hooks/use-auto-scroll.svelte.ts
@allozaur allozaur merged commit dded58b into ggml-org:master May 12, 2026
10 checks passed
@tccybo

tccybo commented May 13, 2026

Copy link
Copy Markdown

@allozaur working well! thanks so much for the fix!

xxmustafacooTR pushed a commit to xxPlayground/llama-cpp-turboquant that referenced this pull request May 13, 2026
…ebKit (ggml-org#22977)

* debug: Scroll/Sticky issues

* fix: UI improvements

* refactor: Remove unneeded logic

* fix: Better logic for initial load of messages
@iridium87

Copy link
Copy Markdown

Looks good here. Thank you!

@allozaur allozaur deleted the 21276-webui-chat-screen-form-box-disappearing branch May 14, 2026 17:08
rsenthilkumar6 pushed a commit to rsenthilkumar6/llama.cpp that referenced this pull request May 19, 2026
…ebKit (ggml-org#22977)

* debug: Scroll/Sticky issues

* fix: UI improvements

* refactor: Remove unneeded logic

* fix: Better logic for initial load of messages
baramofme pushed a commit to baramofme/llama-cpp-turboquant that referenced this pull request May 23, 2026
…ebKit (ggml-org#22977)

* debug: Scroll/Sticky issues

* fix: UI improvements

* refactor: Remove unneeded logic

* fix: Better logic for initial load of messages
baramofme pushed a commit to baramofme/llama-cpp-turboquant that referenced this pull request May 23, 2026
…ebKit (ggml-org#22977)

* debug: Scroll/Sticky issues

* fix: UI improvements

* refactor: Remove unneeded logic

* fix: Better logic for initial load of messages
winstonma pushed a commit to winstonma/llama.cpp that referenced this pull request May 27, 2026
…ebKit (ggml-org#22977)

* debug: Scroll/Sticky issues

* fix: UI improvements

* refactor: Remove unneeded logic

* fix: Better logic for initial load of messages
fewtarius pushed a commit to fewtarius/llama.cpp that referenced this pull request May 30, 2026
…ebKit (ggml-org#22977)

* debug: Scroll/Sticky issues

* fix: UI improvements

* refactor: Remove unneeded logic

* fix: Better logic for initial load of messages
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Misc. bug: webui: ios Safari typing box disappearing

6 participants