Skip to content

Update chat status dashboard based on feedback#313367

Merged
pwang347 merged 7 commits intomainfrom
pawang/updateChatDashboard
Apr 30, 2026
Merged

Update chat status dashboard based on feedback#313367
pwang347 merged 7 commits intomainfrom
pawang/updateChatDashboard

Conversation

@pwang347
Copy link
Copy Markdown
Member

@pwang347 pwang347 commented Apr 29, 2026

image image image

Fixes #313290

Copilot AI review requested due to automatic review settings April 29, 2026 23:10
@pwang347 pwang347 marked this pull request as ready for review April 29, 2026 23:17
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

This PR updates the Chat Status Dashboard UI/UX in the workbench and enhances the Copilot workspace indexing status contribution to better reflect state (including icons), based on feedback.

Changes:

  • Refactors the dashboard to show “Inline Suggestions” and each contributed chat status item as separate collapsible sections.
  • Replaces action-based model/provider option pickers with native inline <select> dropdowns and adds a tri-state per-language override control for inline suggestions enablement.
  • Adds icon metadata to the workspace indexing chat status item and prefixes those icons into the description string.
Show a summary per file
File Description
src/vs/workbench/contrib/chat/browser/chatStatus/media/chatStatus.css Adds styling for contributed section headers/status, section body text, overridden hint, and native inline <select> dropdowns.
src/vs/workbench/contrib/chat/browser/chatStatus/chatStatusDashboard.ts Reworks dashboard rendering (collapsibles, contributed sections), introduces inline selects, and adds a tri-state per-language setting control + status text.
extensions/copilot/src/extension/workspaceChunkSearch/vscode-node/workspaceIndexingStatus.ts Adds optional icons to the indexing status state and includes them in the chat status item description.

Copilot's findings

Comments suppressed due to low confidence (1)

src/vs/workbench/contrib/chat/browser/chatStatus/chatStatusDashboard.ts:514

  • The provider option <select> is also missing a programmatic label. Since option.label is rendered in a separate <span>, screen readers may not associate it with the combobox. Consider wiring them up via <label for>/id or aria-labelledby on the select.
							optionContainer.appendChild($('span.suggest-option-text', undefined, option.label));

							const select = optionContainer.appendChild($('select.inline-select')) as HTMLSelectElement;
							for (const value of option.values) {
  • Files reviewed: 3/3 changed files
  • Comments generated: 9

Comment thread src/vs/workbench/contrib/chat/browser/chatStatus/chatStatusDashboard.ts Outdated
Comment thread src/vs/workbench/contrib/chat/browser/chatStatus/media/chatStatus.css Outdated
Comment thread src/vs/workbench/contrib/chat/browser/chatStatus/chatStatusDashboard.ts Outdated
Comment thread src/vs/workbench/contrib/chat/browser/chatStatus/chatStatusDashboard.ts Outdated
roblourens
roblourens previously approved these changes Apr 29, 2026
@pwang347 pwang347 marked this pull request as draft April 30, 2026 00:09
@pwang347 pwang347 marked this pull request as ready for review April 30, 2026 01:24
@pwang347 pwang347 merged commit 24da6b8 into main Apr 30, 2026
41 of 42 checks passed
@pwang347 pwang347 deleted the pawang/updateChatDashboard branch April 30, 2026 02:19
@vs-code-engineering vs-code-engineering Bot added this to the 1.119.0 milestone Apr 30, 2026
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.

Codebase Semantic Index in Copilot Status Menu

4 participants