Binaryify / OneDark-Pro

Atom's iconic One Dark theme for Visual Studio Code
https://marketplace.visualstudio.com/items?itemName=zhuangtongfa.Material-theme
MIT License
1.52k stars 289 forks source link

Comment is too dark #148

Closed Yensan closed 7 years ago

Yensan commented 7 years ago

When I turn on OneDark-Pro, comment text is too dark to see. Please give a brighter color and make an enough color contrast between the background color and comment text color. vs code default theme "Default Dark+" is better in color contrast of comment/background. Maybe you can just folow the setting of comment text color.

garritfra commented 7 years ago

Fix Request :)

jens1o commented 7 years ago

<< is an idiot.

jens1o commented 7 years ago

tbh, I think the change is too extrem: Having now: image

Incoming: image

garritfra commented 7 years ago

maybe make it a little bit darker again?

jens1o commented 7 years ago

cc @Binaryify for his opinion.

Binaryify commented 7 years ago

@garritfra I think that is good, could you send the PR?

garritfra commented 7 years ago

Done. Relatively new to GH so I don't know if I have done everything correctly.

Yensan commented 7 years ago

Great job ! Thanks all of you.

cc @Binaryify for his opinion.

Maybe he is right, because colors in this theme is designed for token or var in code, but grey is for comment / block comment.

Yensan commented 7 years ago

<<is an idiot.

May you guys think this issue is not necessary. But for a user, although I can edit the color hex code, but I don't want to edit it every time when it updating. Life is short, we need out-of-the-box theme&thing.

jens1o commented 7 years ago

@Yensan Please keep calm. It's not the time to come up with such phrases. Of course we're on it. Remember, that we're doing this in our freetime(and for free). In 20 minutes for example, I need to go to school. You may take a look at custom modifications, that can be located into your user settings.

Yensan commented 7 years ago

Thanks you guys for the theme, and thank you for advice.

Binaryify commented 7 years ago

@Yensan thank you too 🎉

garritfra commented 7 years ago

should this issue be closed?

Binaryify commented 7 years ago

@garritfra I think yes

jens1o commented 7 years ago

@Binaryify image

I think you forgot to update the end and the beginning of a comment. Do you take that job?

Binaryify commented 7 years ago

@jens1o oh, I ignore it, wait

Binaryify commented 7 years ago

@jens1o done