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

Text gets invisible while running mocha #59

Closed
umayr opened this issue Dec 5, 2015 · 11 comments
Closed

Text gets invisible while running mocha #59

umayr opened this issue Dec 5, 2015 · 11 comments

Comments

@umayr
Copy link

umayr commented Dec 5, 2015

screen shot 2015-12-05 at 18 48 57

As seen in the screenshot, some part of the text gets invisible for Solarized Dark theme. It's happening with gulp too.

My iTerm version is 2.1.4.

@kevin-smets
Copy link
Contributor

I fixed this in a gist of mine (I should launch a PRQ for this :) ) -> https://gist.github.com/kevin-smets/8568070#file-solarized-dark-fixed-itermcolors

The link fixes this: by default, the Ansi 8 Color is not readable on the set background for Solarized dark. This value has been replaced by the one from the Solarized high contrast theme.

@mbadolato
Copy link
Owner

| should launch a PRQ for this

👍Please do!

@kevin-smets kevin-smets mentioned this issue Dec 6, 2015
@kevin-smets
Copy link
Contributor

Done :)

Before:
screen shot 2015-12-06 at 20 55 31

After:
screen shot 2015-12-06 at 20 55 58

@umayr
Copy link
Author

umayr commented Dec 6, 2015

@kevin-smets Awesome. But shouldn't it be any base colour rather than blue? Maybe #586e75 or #657b83?

@kevin-smets
Copy link
Contributor

@umayr good point, but that way it does not look like a highlight at all? It looks just like regular output.

(This is #657b83)
screen shot 2015-12-06 at 21 12 40

Just thought high contrast would be the nicest / closest match. Though the rest of the colors are indeed more grey like in Solarized Dark. But in the high contrast version, the highlight versions match the base colors more closely...

@mbadolato what do you think?

@kevin-smets
Copy link
Contributor

Went with Base01, the darkest content color:

screen shot 2015-12-06 at 21 29 04

Also adjusted the PRQ to this.

@mbadolato
Copy link
Owner

Either works for me; no particular preference. I may be one of the few people that doesn't use Solarized Dark 😉

mbadolato added a commit that referenced this issue Dec 6, 2015
@umayr
Copy link
Author

umayr commented Dec 7, 2015

That looks really awesome. Thank you, @kevin-smets.

@kevin-smets
Copy link
Contributor

No problem, now I can simplify my gist :D 👯

@eyaleizenberg
Copy link

I'm still facing this issue, what's the solution here?

screen shot 2017-11-29 at 17 27 34

screen shot 2017-11-29 at 17 28 06

@mbadolato
Copy link
Owner

The issue was reverted (see #62). So other than changing the color manually yourself, or using a tweaked theme (or different altogether), this issue is likely to stay, if you're using something that has output in the affected color. AFAIK, the Solarized theme in here matches the official standard, so it's not something with the theme itself, but rather with the output of the program

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

No branches or pull requests

4 participants