Skip to content

Fix webview tool output disappearing#290055

Merged
mjbvz merged 1 commit intomicrosoft:mainfrom
mjbvz:dev/mjbvz/raw-jellyfish
Jan 24, 2026
Merged

Fix webview tool output disappearing#290055
mjbvz merged 1 commit intomicrosoft:mainfrom
mjbvz:dev/mjbvz/raw-jellyfish

Conversation

@mjbvz
Copy link
Copy Markdown
Collaborator

@mjbvz mjbvz commented Jan 23, 2026

Fixes #289237

Copilot AI review requested due to automatic review settings January 23, 2026 23:57
@vs-code-engineering vs-code-engineering bot added this to the January 2026 milestone Jan 23, 2026
@mjbvz mjbvz enabled auto-merge January 24, 2026 00:02
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Addresses a chat UI regression where tool output rendered via webviews can disappear after the element is detached/hidden and later shown again.

Changes:

  • Reinitialize tool output webviews when the chat content part becomes visible again.
  • Reinitialize tool output webviews when the content part is remounted after virtualization.
  • Pass a remount event from ChatToolInvocationPart down to ChatToolOutputSubPart.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/vs/workbench/contrib/chat/browser/widget/chatContentParts/toolInvocationParts/chatToolOutputPart.ts Hooks tool output webview reinitialization to visibility changes and remount events so output doesn’t disappear after detach/hide.
src/vs/workbench/contrib/chat/browser/widget/chatContentParts/toolInvocationParts/chatToolInvocationPart.ts Adds and wires an onDidRemount event into ChatToolOutputSubPart construction.

@mjbvz mjbvz merged commit 6bf56b0 into microsoft:main Jan 24, 2026
22 of 23 checks passed
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Mar 10, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Mermaid diagrams disappear

3 participants