Skip to content

fix: remove deleted open chat from sidebar folder - #7

Draft
macodev00 wants to merge 1 commit into
devfrom
fix/sidebar-delete-open-chat-folder-31321
Draft

macodev00 wants to merge 1 commit into
devfrom
fix/sidebar-delete-open-chat-folder-31321

Conversation

@macodev00

Copy link
Copy Markdown
Owner

This draft on macodev00/open-webui prepares Closes #31321 for an upstream pull request later. It is not a pull request against open-webui/open-webui.

Summary

Deleting the open chat with the Delete Chat shortcut (Ctrl+Shift+Backspace) removes the chat, but the entry stays under its expanded sidebar folder until a full reload. Clicking that leftover entry opens a chat that no longer exists.

Since open-webui#30165 the shortcut clicks the hidden navbar #delete-chat-button, which confirms through confirmDeleteChat in Chat.svelte. That handler refreshed the recent and pinned lists, but not sidebar folder chat lists. Deleting the same chat from its sidebar row still reloads folder items, and archiving from the navbar already calls refreshFolderChatLists().

Change

After a successful delete, call refreshFolderChatLists() so every open folder reloads its chats. Same call archive and move already use.

Verification

  • Root cause checked on current open-webui/open-webui dev (420b4a279).
  • prettier 3.3.3 leaves this one-line addition unchanged aside from an unrelated pre-existing indent in the same file, which is not part of this commit.
  • ESLint on Chat.svelte reports existing issues only; frontend CI does not run ESLint.
  • The shortcut path was not exercised in a browser in this environment.
Open in Web Open in Cursor 

@macodev00
macodev00 force-pushed the fix/sidebar-delete-open-chat-folder-31321 branch 2 times, most recently from 9e8dc32 to 2d1c5e9 Compare September 28, 2026 06:28
The Delete Chat shortcut confirms through the navbar handler, which
refreshed the recent and pinned lists but not open sidebar folders.
@macodev00
macodev00 force-pushed the fix/sidebar-delete-open-chat-folder-31321 branch from 2d1c5e9 to ec2f04b Compare September 28, 2026 06:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant