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

"Discovering files" icon not responsive to theme #1638

Closed
james-b-yu opened this issue Mar 4, 2018 · 3 comments
Closed

"Discovering files" icon not responsive to theme #1638

james-b-yu opened this issue Mar 4, 2018 · 3 comments
Labels
bug fixed Check the Milestone for the release in which the fix is or will be available. help wanted Can be fixed in the public (open source) repo. Language Service quick fix
Milestone

Comments

@james-b-yu
Copy link

james-b-yu commented Mar 4, 2018

After enjoying the extension for several months now, I have noticed that the "Discovering files" icon in the right-hand side of the status bar does not respond to the theme. This makes it virtually useless in light themes when it becomes virtually invisible.
image
The icon is indeed there as it is hoverable -- the tooltip is still displayed on mouse-over.
image

The icon is of course fully visible in a dark theme:
image

This looks very unprofessional but the fix shouldn't be too difficult as VSCode is already aware of which themes are dark and which are light:
image

As an alternative, an actual progress bar or text display which shows the progress and extent of files indexed/to be indexed could be implemented.

@bobbrow bobbrow added bug Language Service help wanted Can be fixed in the public (open source) repo. quick fix labels Mar 5, 2018
@bobbrow
Copy link
Member

bobbrow commented Mar 5, 2018

I didn't realize we were setting a color for this icon. In any case it looks like VSCode has opened up the statusbar item API to allow for us to link these icons to the theme colors. Should be an easy fix. Thanks for the report.

@sean-mcmanus
Copy link
Contributor

Yeah, we had discussions about this previously -- it was decided to hardcode the colors to the red flame and the white database icon, but I didn't realize a theme had a white status bar. We should probably change this to statusBar.foreground (see https://code.visualstudio.com/docs/getstarted/theme-color-reference ). Not sure if there's any themes with a red status bar we need to handle.

@bobbrow bobbrow added the fixed Check the Milestone for the release in which the fix is or will be available. label Mar 21, 2018
@bobbrow bobbrow added this to the March 2018 milestone Mar 21, 2018
@sean-mcmanus
Copy link
Contributor

Should be fixed with 0.16.1.

@github-actions github-actions bot locked and limited conversation to collaborators Oct 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug fixed Check the Milestone for the release in which the fix is or will be available. help wanted Can be fixed in the public (open source) repo. Language Service quick fix
Projects
None yet
Development

No branches or pull requests

3 participants