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

Turning down the contrast. #19

Closed
wants to merge 1 commit into from
Closed

Conversation

mikeumus
Copy link

The lines for tabs and the dots for spaces where too bright for all Brackets themes so I toned them down to have less contrast and compared it to SublimeText. If you compare the change to SublimeText you'll notice it's closer to SublimeText's contrast after this change as well. Cheers!

The lines for tabs and the dots for spaces where too bright for all Brackets themes so I toned them down to have less contrast and compared it to SublimeText. If you compare the change to SublimeText you'll notice it's closer to SublimeText's contrast after this change as well. Cheers!
@mikeumus
Copy link
Author

Referencing issue #10

@tomByrer
Copy link

tomByrer commented May 1, 2014

too bright for all Brackets themes

My background is very bright:
suggestion-brackets-show-whitespace_css
I've had to brighten the colors so they blended in better. So your patch would have been worse for me.
But I understand you don't want the whitespace to stick out too much. Seems the hipster themes are mostly very dark background (aside from the default, which isn't too bad IMHO).

I wish we could override colors in your own custom theme.

@mikeumus
Copy link
Author

mikeumus commented May 1, 2014

I see. No worries. :)

@tomByrer
Copy link

tomByrer commented May 1, 2014

Yep, you can override with your theme. :)

@mikeumus
Copy link
Author

mikeumus commented May 1, 2014

Already did!, 👍 your main.less actually.

@mikeumus mikeumus closed this May 1, 2014
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

Successfully merging this pull request may close these issues.

None yet

2 participants