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

JSX style is unreadable and too dark #22

Open
dbkaplun opened this issue Nov 19, 2016 · 9 comments
Open

JSX style is unreadable and too dark #22

dbkaplun opened this issue Nov 19, 2016 · 9 comments

Comments

@dbkaplun
Copy link

screen shot 2016-11-19 at 1 04 52 pm
Code in question

@simurai
Copy link
Owner

simurai commented Nov 22, 2016

@dbkaplun Thanks for example code and screenshot.. that's very helpful.

Just another question: What's the language package you are using? I can see the grammar as `JavaScript (JSX).

@dbkaplun
Copy link
Author

I'm using [email protected], is that what you mean? That's the only package I see when I search for "jsx" in the Packages tab of Settings. Maybe it's built-in?

@simurai
Copy link
Owner

simurai commented Nov 22, 2016

[email protected] shouldn't do syntax highlighting. Do you have https://atom.io/packages/language-babel. It has jsx support and is quite popular.

@dbkaplun
Copy link
Author

I have package [email protected], maybe that's it? It's way too dark with .js as well. I'll attach a screenshot when I get the chance.

@simurai
Copy link
Owner

simurai commented Nov 22, 2016

Yes, react is probably it. I'll try it out with your sample code and see why it's so dark.

@dbkaplun
Copy link
Author

Here's some nice dark JS code
screen shot 2016-11-25 at 5 45 42 pm

@simurai
Copy link
Owner

simurai commented Nov 26, 2016

Ok, this should be improved with #21 and published as 2.0.1.

screen shot 2016-11-26 at 10 25 56 pm

@simurai simurai closed this as completed Nov 26, 2016
@dbkaplun
Copy link
Author

Definitely looks a little better, still very dark though (specifically punctuation and builtins like moveTo and Math.sin). Even linesCtx is very dark. It's a shame because this is my favorite theme and I hate having to use other themes. Should I open another issue? Looks like your Atom is significantly redder than mine.. on your screen, beginPath is white and on mine it's lavender. Maybe that has something to do with it?
screen shot 2016-11-26 at 9 20 32 am

@simurai
Copy link
Owner

simurai commented Nov 28, 2016

Looks like your Atom is significantly redder than mine.. on your screen, beginPath is white and on mine it's lavender. Maybe that has something to do with it?

In DuoTone dark you can now change the colors in the settings, see: https://github.com/simurai/duotone-dark-syntax#settings So you could brighten everything.

I'll see about brightening the default since that seems a bit dark.

@simurai simurai reopened this Nov 28, 2016
simurai added a commit to simurai/duotone-dark-syntax that referenced this issue Nov 28, 2016
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