Conversation
bpasero
left a comment
There was a problem hiding this comment.
Seeing a few more copilot icons usage:
Maybe this one needs to be the VS Code icon:
|
|
||
| const welcomeContent: IChatViewWelcomeContent = { | ||
| title: localize('expChatTitle', 'Welcome to Copilot'), | ||
| title: localize('expChatTitle', 'Ask, Edit, and Build'), |
There was a problem hiding this comment.
I think @cwebster-99 suggested Build with agent mode
There was a problem hiding this comment.
Pull Request Overview
This PR replaces various copilot-related icons with the chat sparkle icon and removes trailing periods from welcome titles. The changes standardize the chat interface to use a consistent sparkle icon instead of the previous copilot icons, while also cleaning up punctuation in user-facing titles.
Key changes:
- Replaced copilot icons with chat sparkle icon throughout the chat interface
- Removed trailing periods from chat mode titles for consistency
- Updated welcome screen title and icon
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/vs/workbench/contrib/chat/browser/chatWidget.ts | Updated icon selection logic and welcome titles, removing periods and changing copilot references to chat sparkle |
| src/vs/workbench/contrib/chat/browser/chatListRenderer.ts | Changed default fallback icon from copilot to chat sparkle |
|
|
||
| const welcomeContent: IChatViewWelcomeContent = { | ||
| title: localize('expChatTitle', 'Welcome to Copilot'), | ||
| title: localize('expChatTitle', 'Build with agent mode'), |
There was a problem hiding this comment.
The welcome screen title is now duplicating the agent mode title. This change makes the welcome screen title identical to the agent mode title on line 1366, which could be confusing for users. The welcome screen should have a more generic title like 'Welcome to Chat' or retain the original 'Welcome to Copilot' title.
| title: localize('expChatTitle', 'Build with agent mode'), | |
| title: localize('expChatTitle', 'Welcome to Chat'), |
Removed frog icon and revised welcome title.