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

Path gets blacked on hover in terminal if hoverHighlightBackground is opaque #97521

Closed
AnilSonix opened this issue May 11, 2020 · 11 comments · Fixed by #98974
Closed

Path gets blacked on hover in terminal if hoverHighlightBackground is opaque #97521

AnilSonix opened this issue May 11, 2020 · 11 comments · Fixed by #98974
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug terminal General terminal issues that don't fall under another label verified Verification succeeded
Milestone

Comments

@AnilSonix
Copy link

  • VSCode Version: 1.46.0-Insider
  • OS Version: Ubuntu 20.04

Steps to Reproduce:

  1. Open any folder or workspace
  2. Open terminal
  3. Navigate to any nested directories.
  4. Hover over the path in the terminal.
  5. The path becomes black.

Does this issue occur when all extensions are disabled?: No

normal
error

@AnilSonix AnilSonix changed the title Eleemnt gets blacked on hover in terminal Path gets blacked on hover in terminal May 11, 2020
@Tyriar
Copy link
Member

Tyriar commented May 11, 2020

I'm guessing you have a non-transparent color set as editor.hoverHighlightBackground in your theme.

@Tyriar Tyriar added bug Issue identified by VS Code Team member as probable bug terminal General terminal issues that don't fall under another label labels May 11, 2020
@Tyriar Tyriar added this to the May 2020 milestone May 11, 2020
@Tyriar Tyriar self-assigned this May 11, 2020
@AnilSonix
Copy link
Author

How can I override it ?

@Tyriar
Copy link
Member

Tyriar commented May 14, 2020

This setting:

"workbench.colorCustomizations": {
  "editor.hoverHighlightBackground": "#ff000080"
}

@AnilSonix
Copy link
Author

Thank you 👍

@dsifford
Copy link

Following the instructions you provided @AnilSonix I still cannot reproduce this.

@Tyriar Tyriar changed the title Path gets blacked on hover in terminal Path gets blacked on hover in terminal if hoverHighlightBackground is opaque May 26, 2020
@Tyriar Tyriar reopened this May 26, 2020
@Tyriar
Copy link
Member

Tyriar commented May 26, 2020

The proper fix for this to draw the highlight under the text probably needs xtermjs/xterm.js#1852 to happen. We could do a workaround to make the color partially transparent if it's detected as opaque.

@Tyriar Tyriar added the help wanted Issues identified as good community contribution opportunities label May 26, 2020
@dsifford
Copy link

Cool, thanks for the update @Tyriar

Just went ahead and merged @AnilSonix's workaround in dracula theme for the time being. I was finally able to repro on my end 👍

@ChrisPapp
Copy link
Contributor

Can I work on this?

@Tyriar
Copy link
Member

Tyriar commented Jun 1, 2020

@ChrisPapp I'll have to do a quick fix on this to squeeze it in for this release.

@Tyriar Tyriar removed the help wanted Issues identified as good community contribution opportunities label Jun 1, 2020
@ChrisPapp
Copy link
Contributor

@Tyriar I have made a quick fix myself. I can push it now and you can take a look.

@Tyriar
Copy link
Member

Tyriar commented Jun 1, 2020

@ChrisPapp if it's ready now, sure! 👍

@mjbvz mjbvz added the verified Verification succeeded label Jun 5, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Jul 16, 2020
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 terminal General terminal issues that don't fall under another label verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants