-
Notifications
You must be signed in to change notification settings - Fork 187
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
Blackness #9
Comments
Thanks for trying it out :) I believe the background color is applied on npm's side. I've looked through their code a little bit to see if I can find exactly where it's coming from but unfortunately that's no easy task.. However, here is the output in a Terminal window using the default "Basic" theme: As you can see the background color for each module is still black, so I don't think it's related to the one-dark-terminal theme but rather npm itself, as that is how they've chosen to display tree views. |
Ah thanks, nice spot. |
That's a good point.. I'd be a bit hesitant to not have "Black" be "#000000", but if it won't cause any conflicts elsewhere I see no reason why not. Do you know where else this black color may be used? |
Good enough for me, I'll post an update tonight. Thanks for the tip! |
This won't be solvable for the one-light theme, however. I'm not going to use |
I run
npm install
frequently, and it looks like this:However, I would expect it to look like this:
The text was updated successfully, but these errors were encountered: