From 1e56b4dd62348e9de172829e3a25cff0c522d20b Mon Sep 17 00:00:00 2001 From: Hugo Locurcio Date: Sat, 15 Dec 2018 18:06:26 +0100 Subject: [PATCH] Tweak editor status colors (success, warning, error) Some of the previous colors were "overbright" in the sense that some of their components were above 1, causing font anti-aliasing to look bad. These new colors should be easier on the eyes while fitting better with the rest of the editor's color palette. --- editor/editor_themes.cpp | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/editor/editor_themes.cpp b/editor/editor_themes.cpp index 36053d75348..eacf2ee1b41 100644 --- a/editor/editor_themes.cpp +++ b/editor/editor_themes.cpp @@ -351,18 +351,18 @@ Ref create_editor_theme(const Ref p_theme) { theme->set_color("mono_color", "Editor", mono_color); - Color success_color = accent_color.linear_interpolate(Color(0.2, 1, 0.2), 0.6) * 1.2; - Color warning_color = accent_color.linear_interpolate(Color(1, 1, 0), 0.7) * 1.0; - Color error_color = accent_color.linear_interpolate(Color(1, 0, 0), 0.8) * 1.7; + Color success_color = Color(0.45, 0.95, 0.5); + Color warning_color = Color(1, 0.87, 0.4); + Color error_color = Color(1, 0.47, 0.42); Color property_color = font_color.linear_interpolate(Color(0.5, 0.5, 0.5), 0.5); if (!dark_theme) { - // yellow on white themes is a P.I.T.A. - warning_color = accent_color.linear_interpolate(Color(0.9, 0.7, 0), 0.9); - warning_color = warning_color.linear_interpolate(mono_color, 0.2); - success_color = success_color.linear_interpolate(mono_color, 0.2); - error_color = error_color.linear_interpolate(mono_color, 0.2); + // Darken some colors to be readable on a light background + success_color = success_color.linear_interpolate(mono_color, 0.35); + warning_color = warning_color.linear_interpolate(mono_color, 0.35); + error_color = error_color.linear_interpolate(mono_color, 0.25); } + theme->set_color("success_color", "Editor", success_color); theme->set_color("warning_color", "Editor", warning_color); theme->set_color("error_color", "Editor", error_color);