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

Blackness #9

Closed
amk221 opened this issue Aug 10, 2016 · 7 comments
Closed

Blackness #9

amk221 opened this issue Aug 10, 2016 · 7 comments
Assignees

Comments

@amk221
Copy link

amk221 commented Aug 10, 2016

I run npm install frequently, and it looks like this:

However, I would expect it to look like this:

@nathanbuchar
Copy link
Owner

nathanbuchar commented Aug 10, 2016

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:

image

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.

@amk221
Copy link
Author

amk221 commented Aug 10, 2016

Ah thanks, nice spot.

@amk221 amk221 closed this as completed Aug 10, 2016
@amk221
Copy link
Author

amk221 commented Aug 10, 2016

Apologies if this a daft question, but why is black even a part of the One Dark theme?
If I change Black to the Background colour provided by One Dark theme, it looks fine as per my original screenshot.

screen shot 2016-08-10 at 21 43 20

@nathanbuchar
Copy link
Owner

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?

@amk221
Copy link
Author

amk221 commented Aug 10, 2016

I'm afraid I don't know enough about terminal colours... but Solarized Dark doesn't have Black

screen shot 2016-08-10 at 21 57 41

@nathanbuchar
Copy link
Owner

Good enough for me, I'll post an update tonight. Thanks for the tip!

@nathanbuchar
Copy link
Owner

This won't be solvable for the one-light theme, however. I'm not going to use #FFFFFF as "Black" haha, but for one-dark it shouldn't be an issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants