Skip to content

Commit 06faefc

Browse files
committed
Merge pull request #112313 from passivestar/separator-contrast
Improve visibility of separators in modern theme
2 parents 2132ba8 + 0cbb788 commit 06faefc

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

editor/themes/theme_modern.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -175,7 +175,7 @@ void ThemeModern::populate_shared_styles(const Ref<EditorTheme> &p_theme, Editor
175175
p_config.selection_color = p_config.accent_color * Color(1, 1, 1, 0.4);
176176
p_config.disabled_border_color = p_config.mono_color.inverted().lerp(p_config.base_color, 0.7);
177177
p_config.disabled_bg_color = p_config.mono_color.inverted().lerp(p_config.base_color, 0.9);
178-
p_config.separator_color = p_config.dark_theme ? Color(0, 0, 0, 0.3) : Color(0, 0, 0, 0.1);
178+
p_config.separator_color = p_config.dark_theme ? Color(0, 0, 0, 0.4) : Color(0, 0, 0, 0.2);
179179

180180
p_theme->set_color("selection_color", EditorStringName(Editor), p_config.selection_color);
181181
p_theme->set_color("disabled_border_color", EditorStringName(Editor), p_config.disabled_border_color);
@@ -991,7 +991,7 @@ void ThemeModern::populate_standard_styles(const Ref<EditorTheme> &p_theme, Edit
991991

992992
p_theme->set_stylebox(SceneStringName(hover), "PopupMenu", p_config.flat_button_hover);
993993

994-
Ref<StyleBoxLine> style_popup_separator = EditorThemeManager::make_line_stylebox(Color(0, 0, 0, p_config.dark_theme ? 0.2 : 0.1), Math::round(2 * EDSCALE), p_config.base_margin * -1.5 * EDSCALE, p_config.base_margin * -1.5 * EDSCALE);
994+
Ref<StyleBoxLine> style_popup_separator = EditorThemeManager::make_line_stylebox(p_config.mono_color * Color(1, 1, 1, p_config.dark_theme ? 0.075 : 0.125), Math::round(2 * EDSCALE), p_config.base_margin * -2 * EDSCALE, p_config.base_margin * -2 * EDSCALE);
995995

996996
p_theme->set_stylebox("separator", "PopupMenu", style_popup_separator);
997997
p_theme->set_stylebox("labeled_separator_left", "PopupMenu", style_popup_separator);

0 commit comments

Comments
 (0)