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

More Readable Color for Function Call ? #7

Closed
GulajavaMinistudio opened this issue Jan 1, 2020 · 6 comments
Closed

More Readable Color for Function Call ? #7

GulajavaMinistudio opened this issue Jan 1, 2020 · 6 comments

Comments

@GulajavaMinistudio
Copy link

Hello, thanks for good theme. Recently, i try this theme in one of our project with Vue JS with Vue CLI in VS Code. But it resulted, green color for function name is not easy readable. You can take a look in this screenshot.

This is for your Gruvbox Material Theme
Screenshot_2020-01-01_19-12-56

Compare it with other theme, such as Gruvbox Original
Screenshot_2020-01-01_19-12-18

Or with Monokai Pro theme
Screenshot_2020-01-01_19-13-53

As you can see, your theme with Green color for function name in "methods" in Vue Component look not good and not easy readable. Maybe you can change this color for more green or more vivid green color, so it easy readable ?

@sainnhe
Copy link
Owner

sainnhe commented Jan 1, 2020

I haven't optimize vue for this theme yet, I'll do that later.

@sainnhe
Copy link
Owner

sainnhe commented Jan 2, 2020

Which syntax highlighting extension are you using? I've tried both jcbuisson.vue and liuji-jim.vue but still cannot reproduce.

demo1

@sainnhe
Copy link
Owner

sainnhe commented Jan 2, 2020

This issue may be duplicated with #6.

I have to say that I still keep my point, Babel JavaScript is very bad. Microsoft has done a very great job on javascript syntax highlighting which is buildin in vscode, but this extension makes things terrible, I think not installing it is better than installing it.

I once considered setting gruvbox-material to conflict with this extension, but in the end I decided to remind users not to install this extension in the README.

Still, I compromised. A theme extension should do its best to be compatible with the syntax extensions, no matter how bad the syntax extensions are. I've basically optimized this extension via 326143d.

@GulajavaMinistudio
Copy link
Author

GulajavaMinistudio commented Jan 2, 2020

Oh I'm sorry, I'm using Babel JavaScript syntax highlighting too, I need this extension for other project that using React. Maybe i should disabled it when using your theme. Thank you for workaround

@sainnhe
Copy link
Owner

sainnhe commented Jan 2, 2020

You don't need to disable it now, this problem should be fixed. You can update this theme and get it a try.

@GulajavaMinistudio
Copy link
Author

Thank you, i have been updated gruvbox material extension, and this works better than before when using in Vue JS Project. Now this theme is more readable color now.

Screenshot_2020-01-02_10-04-33

You can close this issue, as it solved.

@sainnhe sainnhe closed this as completed Jan 2, 2020
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

2 participants