Skip to content

Commit

Permalink
Define DEFAULT_IDENTIFIER to only have foreground
Browse files Browse the repository at this point in the history
Since DEFAULT_IDENTIFIER was not defined it inherited its colors
from TEXT, which is where the default background is defined.

This became a problem when identifiers where a part of a template
language snippet, because they would not get the DEFAULT_TEMPLATE_LANGUAGE_COLOR
background but the default background.

We now define DEFAULT_IDENTIFIER with only a foreground color.

Fixes jkaving#95 and fixes jkaving#101
  • Loading branch information
jkaving committed Jul 17, 2015
1 parent 967edb7 commit 1abcc43
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 0 deletions.
5 changes: 5 additions & 0 deletions Solarized Dark.icls
Original file line number Diff line number Diff line change
Expand Up @@ -751,6 +751,11 @@
<option name="FOREGROUND" value="dc322f" />
</value>
</option>
<option name="DEFAULT_IDENTIFIER">
<value>
<option name="FOREGROUND" value="839496" />
</value>
</option>
<option name="DEFAULT_INSTANCE_FIELD">
<value>
<option name="FOREGROUND" value="6c71c4" />
Expand Down
5 changes: 5 additions & 0 deletions Solarized Light.icls
Original file line number Diff line number Diff line change
Expand Up @@ -759,6 +759,11 @@
<option name="FOREGROUND" value="dc322f" />
</value>
</option>
<option name="DEFAULT_IDENTIFIER">
<value>
<option name="FOREGROUND" value="586e75" />
</value>
</option>
<option name="DEFAULT_INSTANCE_FIELD">
<value>
<option name="FOREGROUND" value="6c71c4" />
Expand Down
Binary file modified settings.jar
Binary file not shown.

0 comments on commit 1abcc43

Please sign in to comment.