Skip to content

Commit

Permalink
FIX: update nord theme to have a base color for NameOther
Browse files Browse the repository at this point in the history
Right now there is no value for NameOther making .chroma .nx to be empty.

Because of that there are cases in which the color is wrongly used because it's not overwritten (for eg: when trying to use a light theme as the main one, and Nord under prefers-color-scheme: dark .

This MR sets the color to match the one from the theme preview.
  • Loading branch information
CoolGoose authored and alecthomas committed Apr 30, 2023
1 parent 3c21942 commit f720d5a
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion styles/nord.xml
Expand Up @@ -15,6 +15,7 @@
<entry type="NameFunction" style="#88c0d0"/>
<entry type="NameLabel" style="#8fbcbb"/>
<entry type="NameNamespace" style="#8fbcbb"/>
<entry type="NameOther" style="#d8dee9"/>
<entry type="NameTag" style="#81a1c1"/>
<entry type="NameVariable" style="#d8dee9"/>
<entry type="LiteralString" style="#a3be8c"/>
Expand All @@ -41,4 +42,4 @@
<entry type="GenericSubheading" style="bold #88c0d0"/>
<entry type="GenericTraceback" style="#bf616a"/>
<entry type="TextWhitespace" style="#d8dee9"/>
</style>
</style>

0 comments on commit f720d5a

Please sign in to comment.