Skip to content

fix placeholder not showing up + inline chat progress border#312979

Merged
justschen merged 1 commit intomainfrom
justin/klefki
Apr 28, 2026
Merged

fix placeholder not showing up + inline chat progress border#312979
justschen merged 1 commit intomainfrom
justin/klefki

Conversation

@justschen
Copy link
Copy Markdown
Collaborator

  1. fixes placeholders not showing up
  2. fixes progress border not rendering on inline chat

cc @mrleemurray #312337 breaks the placeholders of inline, terminal, and quickchat.

Copilot AI review requested due to automatic review settings April 28, 2026 08:18
@justschen justschen added this to the 1.118.0 milestone Apr 28, 2026
@justschen justschen added the ~release-cherry-pick Trigger: cherry-pick this PR to the latest release branch label Apr 28, 2026
@justschen justschen enabled auto-merge (squash) April 28, 2026 08:19
@vs-code-engineering
Copy link
Copy Markdown
Contributor

This PR will be automatically cherry-picked to release/1.118 when merged.

@github-actions
Copy link
Copy Markdown
Contributor

blocks-ci screenshots changed

Replace the contents of test/componentFixtures/blocks-ci-screenshots.md with:

Updated blocks-ci-screenshots.md
<!-- auto-generated by CI — do not edit manually -->

#### editor/codeEditor/CodeEditor/Dark
![screenshot](https://hediet-screenshots.azurewebsites.net/images/cb32a3e854b5734fe5aaca2318f2e0a42ee821b05ea97883ea42c5ba95edb3c3)

#### editor/codeEditor/CodeEditor/Light
![screenshot](https://hediet-screenshots.azurewebsites.net/images/42624fbba5e0db7f32c224b5eb9c5dd3b08245697ae2e7d2a88be0d7c287129b)

#### editor/inlineChatZoneWidget/InlineChatZoneWidget/Dark
![screenshot](https://hediet-screenshots.azurewebsites.net/images/4baed4b05eeb8aadc8bc2dc633abfc87054cdeea4c35b7a655766063bcfedfed)

#### editor/inlineChatZoneWidget/InlineChatZoneWidget/Light
![screenshot](https://hediet-screenshots.azurewebsites.net/images/b9d098d8ccc2bdb6b816537f14b1f1f3c10dadf294fc0b28d03d6e5a34eaf4c8)

#### editor/inlineChatZoneWidget/InlineChatZoneWidgetTerminated/Dark
![screenshot](https://hediet-screenshots.azurewebsites.net/images/4baed4b05eeb8aadc8bc2dc633abfc87054cdeea4c35b7a655766063bcfedfed)

#### editor/inlineChatZoneWidget/InlineChatZoneWidgetTerminated/Light
![screenshot](https://hediet-screenshots.azurewebsites.net/images/b9d098d8ccc2bdb6b816537f14b1f1f3c10dadf294fc0b28d03d6e5a34eaf4c8)

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

Fixes regressions introduced by placeholder truncation work by scoping the container-query-based placeholder styling to non-compact chat inputs, and adjusts progress-border behavior in inline chat contexts.

Changes:

  • Scope container-type: inline-size and placeholder truncation CSS to non-compact chat inputs to avoid breaking placeholder rendering in compact contexts.
  • Update progress-border enabling logic to exclude inline chat widgets.
Show a summary per file
File Description
src/vs/workbench/contrib/chat/browser/widget/media/chat.css Restricts container-query placeholder truncation styling to non-compact inputs to prevent zero-width placeholder rendering in compact modes.
src/vs/workbench/contrib/chat/browser/widget/chatWidget.ts Changes when the “working/progress border” class is applied, now gating it off for inline chat.

Copilot's findings

  • Files reviewed: 2/2 changed files
  • Comments generated: 1

Comment thread src/vs/workbench/contrib/chat/browser/widget/chatWidget.ts
@justschen justschen merged commit ba3d653 into main Apr 28, 2026
29 of 30 checks passed
@justschen justschen deleted the justin/klefki branch April 28, 2026 08:41
@vs-code-engineering vs-code-engineering Bot added release-cherry-pick Automated cherry-pick between release and main branches and removed ~release-cherry-pick Trigger: cherry-pick this PR to the latest release branch labels Apr 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-cherry-pick Automated cherry-pick between release and main branches

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants