Merge pull request #58731 from Calinou/editor-theme-clamp-base-colors

Clamp the editor theme's base colors to avoid various issues
This commit is contained in:
Rémi Verschelde 2022-03-04 09:46:10 +01:00 committed by GitHub
commit 39dc25376e
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -411,9 +411,11 @@ Ref<Theme> create_editor_theme(const Ref<Theme> p_theme) {
// Colors
bool dark_theme = EditorSettings::get_singleton()->is_dark_theme();
const Color dark_color_1 = base_color.lerp(Color(0, 0, 0, 1), contrast);
const Color dark_color_2 = base_color.lerp(Color(0, 0, 0, 1), contrast * 1.5);
const Color dark_color_3 = base_color.lerp(Color(0, 0, 0, 1), contrast * 2);
// Ensure base colors are in the 0..1 luminance range to avoid 8-bit integer overflow or text rendering issues.
// Some places in the editor use 8-bit integer colors.
const Color dark_color_1 = base_color.lerp(Color(0, 0, 0, 1), contrast).clamp();
const Color dark_color_2 = base_color.lerp(Color(0, 0, 0, 1), contrast * 1.5).clamp();
const Color dark_color_3 = base_color.lerp(Color(0, 0, 0, 1), contrast * 2).clamp();
const Color background_color = dark_color_2;