search

jkaving / intellij-colors-solarized

Solarized Colorscheme for IntelliJ IDEA
3.26k stars 429 forks source link

PhpStorm 2017.1.1 multiple color elements are shown in blue now #147

Closed aik099 closed 7 years ago

aik099 commented 7 years ago

Before: 2017-04-07_1605

After: 2017-04-07_1604

Not sure if the change was intended or just a bug in PhpStorm 2017.1.1, but having both property name and braces in blue makes array key access constructs less readable.

jkaving commented 7 years ago

This change was intended - and was done in pull request #125 (with a lot of discussion in pull request #124). The colors for brackets, braces and parentheses was removed for the different languages, so it should be easy to change them everywhere by changing the Language Default setting.

aik099 commented 7 years ago

This change was intended - and was done in pull request #125 (with a lot of discussion in pull request #124).

I see. And the class property blue color is part of #58 change.

The colors for brackets, braces and parentheses was removed for the different languages, so it should be easy to change them everywhere by changing the Language Default setting.

Yes, but then language default for braces ([ and ]) should be set to red, because it was set to inherited in Java/PHP.