Unify and fix Solarized token colors

This PR contains a few more fixes for both Solarized themes:
1. The token color definitions have diverged a bit, unify them.
   Specifically, this makes both themes use the same colors for the same
   tokens (only affects diff headers and class names) and harmonizes a
   few scopes.
2. `#D30102` is not a Solarized color, replace with `#DC322F` (red).
3. `#219186` is not a Solarized color, replace with `#859900` (green).
This commit is contained in:
Michael Kuhn
2022-02-11 22:48:18 +01:00
parent 06528bf65d
commit cffd1ecb7a
2 changed files with 31 additions and 25 deletions
@@ -7,7 +7,10 @@
}
},
{
"scope": ["meta.embedded", "source.groovy.embedded"],
"scope": [
"meta.embedded",
"source.groovy.embedded"
],
"settings": {
"foreground": "#839496"
}
@@ -31,7 +34,7 @@
"name": "Regexp",
"scope": "string.regexp",
"settings": {
"foreground": "#D30102"
"foreground": "#DC322F"
}
},
{
@@ -100,7 +103,7 @@
"punctuation.section.embedded.end"
],
"settings": {
"foreground": "#D30102"
"foreground": "#DC322F"
}
},
{
@@ -177,12 +180,15 @@
"name": "Continuation",
"scope": "punctuation.separator.continuation",
"settings": {
"foreground": "#D30102"
"foreground": "#DC322F"
}
},
{
"name": "Library constant",
"scope": "support.constant",
"scope": [
"support.constant",
"support.variable"
],
"settings": {}
},
{
@@ -211,7 +217,7 @@
"name": "Invalid",
"scope": "invalid",
"settings": {
"foreground": "#D30102"
"foreground": "#DC322F"
}
},
{
@@ -222,7 +228,7 @@
],
"settings": {
"fontStyle": "italic",
"foreground": "#E0EDDD"
"foreground": "#268BD2"
}
},
{
@@ -230,7 +236,7 @@
"scope": "markup.deleted",
"settings": {
"fontStyle": "",
"foreground": "#dc322f"
"foreground": "#DC322F"
}
},
{
@@ -238,14 +244,14 @@
"scope": "markup.changed",
"settings": {
"fontStyle": "",
"foreground": "#cb4b16"
"foreground": "#CB4B16"
}
},
{
"name": "diff: inserted",
"scope": "markup.inserted",
"settings": {
"foreground": "#219186"
"foreground": "#859900"
}
},
{