Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Terminal: Inline chat recommendation is too distracting #213647

Closed
aeschli opened this issue May 28, 2024 · 6 comments · Fixed by #213783 or #214173
Closed

Terminal: Inline chat recommendation is too distracting #213647

aeschli opened this issue May 28, 2024 · 6 comments · Fixed by #213783 or #214173
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders polish Cleanup and polish issue terminal-chat Issues related to the terminal chat widget under-discussion Issue is under discussion for relevance, priority, approach verified Verification succeeded
Milestone

Comments

@aeschli
Copy link
Contributor

aeschli commented May 28, 2024

Version: 1.90.0-insider
Commit: 50f2b2e
Date: 2024-05-28T05:49:03.008Z
Electron: 29.4.0
ElectronBuildId: 9593362
Chromium: 122.0.6261.156
Node.js: 20.9.0
V8: 12.2.281.27-electron.0
OS: Linux x64 5.14.0-1048-oem

  • Have copilot installed and open the terminal

image

The inline chat recommendation uses the same colors as the terminal prompt and I find it too distracting.
Would be better if it uses the color that ghost text in the editor uses.

@Tyriar
Copy link
Member

Tyriar commented May 28, 2024

Colors look very close to the editor for Dark Modern:

image

I notice they're not italic though, if the colors differ we should make them align (is the gray not as dark?)

@Tyriar Tyriar added this to the May 2024 milestone May 28, 2024
@Tyriar
Copy link
Member

Tyriar commented May 29, 2024

Yes we need to align colors:

image

@Tyriar Tyriar added bug Issue identified by VS Code Team member as probable bug terminal-chat Issues related to the terminal chat widget labels May 29, 2024
Tyriar added a commit that referenced this issue May 29, 2024
@vscodenpa vscodenpa added the unreleased Patch has not yet been released in VS Code Insiders label May 29, 2024
@aeschli
Copy link
Contributor Author

aeschli commented May 29, 2024

IMO just aligning with the colors that are used in the empty editor will not help much.
The empty editor doesn't have any other content, so the hint doesn't conflict with regular content.

We should use the inline-hint colors instead, that are designed to go together with regular text.

@Tyriar
Copy link
Member

Tyriar commented May 29, 2024

@aeschli it's more impactful than that not just to align with the editor but because it doesn't show very often. I think the conditions are that it must be the only terminal in addition to no other commands being run.

Without the blue:

image

Using editorGhostText.foreground:

image

@Tyriar
Copy link
Member

Tyriar commented May 29, 2024

Put a poll out in #ux

@Tyriar Tyriar reopened this May 29, 2024
@vscodenpa vscodenpa removed the unreleased Patch has not yet been released in VS Code Insiders label May 29, 2024
@Tyriar
Copy link
Member

Tyriar commented May 29, 2024

Bunch of discussion happening, we'll enable only on Insiders for this release.

Tyriar added a commit that referenced this issue May 29, 2024
@Tyriar Tyriar modified the milestones: May 2024, June 2024 May 29, 2024
@Tyriar Tyriar added under-discussion Issue is under discussion for relevance, priority, approach and removed bug Issue identified by VS Code Team member as probable bug labels May 29, 2024
meganrogge added a commit that referenced this issue Jun 3, 2024
meganrogge added a commit that referenced this issue Jun 3, 2024
@meganrogge meganrogge added bug Issue identified by VS Code Team member as probable bug polish Cleanup and polish issue labels Jun 3, 2024
meganrogge added a commit that referenced this issue Jun 4, 2024
@vscodenpa vscodenpa added unreleased Patch has not yet been released in VS Code Insiders insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels Jun 4, 2024
@aeschli aeschli added the verified Verification succeeded label Jun 26, 2024
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Jul 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders polish Cleanup and polish issue terminal-chat Issues related to the terminal chat widget under-discussion Issue is under discussion for relevance, priority, approach verified Verification succeeded
Projects
None yet
4 participants