From 198c36811e7ce1e552be7b1cb56ae62d4c399858 Mon Sep 17 00:00:00 2001 From: Marshall Bowers Date: Mon, 24 Feb 2025 11:12:15 -0500 Subject: [PATCH] Change `@variable` color in Gruvbox themes to be less intense (#25464) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This PR changes the color used for `@variable` syntax highlights in the Gruvbox themes to be less intense. We now use the same color as `editor.foreground`. | Language | Before | After | | -------- | ----------------------------------------------------------------------------------------------------------------------------------------------------- | ----------------------------------------------------------------------------------------------------------------------------------------------------- | | Rust | Screenshot 2025-02-24 at 10 08 41 AM | Screenshot 2025-02-24 at 10 55 18 AM | | Python | Screenshot 2025-02-24 at 10 08 38 AM | Screenshot 2025-02-24 at 10 55 20 AM | In #25333 and #25331 the highlight used for identifiers in Rust and Python, respectively, was changed to `@variable`, which resulted in the intense colors you see in the "Before" screenshots above. We considered reverting the highlight query changes to those languages, but after taking a look at our other languages, they already use similar queries. Instead we're adjusting the theme to make these cases less visually intense. Release Notes: - Gruvbox themes: Changed the color used for `@variable` syntax highlights to be less intense. --- assets/themes/gruvbox/gruvbox.json | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/assets/themes/gruvbox/gruvbox.json b/assets/themes/gruvbox/gruvbox.json index 958cf4797dc6f4..f1a6b5c519ba2f 100644 --- a/assets/themes/gruvbox/gruvbox.json +++ b/assets/themes/gruvbox/gruvbox.json @@ -379,7 +379,7 @@ "font_weight": null }, "variable": { - "color": "#83a598ff", + "color": "#ebdbb2ff", "font_style": null, "font_weight": null }, @@ -767,7 +767,7 @@ "font_weight": null }, "variable": { - "color": "#83a598ff", + "color": "#ebdbb2ff", "font_style": null, "font_weight": null }, @@ -1155,7 +1155,7 @@ "font_weight": null }, "variable": { - "color": "#83a598ff", + "color": "#ebdbb2ff", "font_style": null, "font_weight": null }, @@ -1543,7 +1543,7 @@ "font_weight": null }, "variable": { - "color": "#066578ff", + "color": "#282828ff", "font_style": null, "font_weight": null }, @@ -1931,7 +1931,7 @@ "font_weight": null }, "variable": { - "color": "#066578ff", + "color": "#282828ff", "font_style": null, "font_weight": null }, @@ -2319,7 +2319,7 @@ "font_weight": null }, "variable": { - "color": "#066578ff", + "color": "#282828ff", "font_style": null, "font_weight": null },