Skip to content

ui: add close button to UI toasts - #28246

Merged
ServeurpersoCom merged 1 commit into
ggml-org:masterfrom
agustinmista:toast-close-button
Sep 22, 2026
Merged

ServeurpersoCom merged 1 commit into
ggml-org:masterfrom
agustinmista:toast-close-button

Conversation

@agustinmista

Copy link
Copy Markdown
Contributor

Overview

This PR tweaks the Toaster element to include a close button.

These toasts often cover other UI elements like the model selector, and this change avoids having to wait for them to disappear on their own (e.g. after a load failure).

image

Requirements

@agustinmista
agustinmista requested a review from a team as a code owner September 2, 2026 13:03
@agustinmista agustinmista changed the title Add close button to UI toasts ui: add close button to UI toasts Sep 2, 2026
@ServeurpersoCom

Copy link
Copy Markdown
Contributor

I need to check something:

ApproveCI

@ServeurpersoCom

Copy link
Copy Markdown
Contributor

Trying a close/reopen to retrigger the CI, the pending runs reference a workflow file that was removed from master yesterday.

@ServeurpersoCom

Copy link
Copy Markdown
Contributor

It worked, the CI is running again!

@ServeurpersoCom ServeurpersoCom left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ServeurpersoCom

Copy link
Copy Markdown
Contributor

Thanks for the rebase, it was not needed though, I would have merged it anyway since it was green, I just took advantage of the small change to check a GitHub CI corner case:)

This commit tweaks the Toaster element to include a close button.

These toasts often cover other UI elements like the model selector, and
this change avoids having to wait for them to disappear on their own
(e.g. after a load failure).
@ServeurpersoCom
ServeurpersoCom merged commit 8cfc315 into ggml-org:master Sep 22, 2026
4 checks passed
@agustinmista

Copy link
Copy Markdown
Contributor Author

Hey @ServeurpersoCom! Sorry, I was trying to fix the linting issue and I think I stepped on your toes 😅

@ServeurpersoCom

Copy link
Copy Markdown
Contributor

Hey @ServeurpersoCom! Sorry, I was trying to fix the linting issue and I think I stepped on your toes 😅

I saw it as entirely green on my end. But just as I was about to click, the rebase reset the CI:)

LadislavSopko pushed a commit to 0ics-srls/llama.cpp that referenced this pull request Oct 5, 2026
This commit tweaks the Toaster element to include a close button.

These toasts often cover other UI elements like the model selector, and
this change avoids having to wait for them to disappear on their own
(e.g. after a load failure).
frostyautumnleaf pushed a commit to frostyautumnleaf/llama.cpp that referenced this pull request Oct 5, 2026
This commit tweaks the Toaster element to include a close button.

These toasts often cover other UI elements like the model selector, and
this change avoids having to wait for them to disappear on their own
(e.g. after a load failure).
edwardyoon pushed a commit to edwardyoon/focus-llama that referenced this pull request Oct 7, 2026
This commit tweaks the Toaster element to include a close button.

These toasts often cover other UI elements like the model selector, and
this change avoids having to wait for them to disappear on their own
(e.g. after a load failure).

(cherry picked from commit 8cfc315)
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.

3 participants