Skip to content

Conversation

@DeeJayLSP
Copy link
Contributor

@DeeJayLSP DeeJayLSP commented Nov 5, 2025

The color was calculated wrongly due to being based on the Classic style.

It remains too dark if the editor main color is increased before this change, when it should be a highlight.

#7f7f7f is the brightest color the editor can have before considering it as a Light theme.

Color Before After
#252525 before_dark after_dark
#7f7f7f before_7f7f7f after_7f7f7f

@DeeJayLSP DeeJayLSP requested review from a team as code owners November 5, 2025 16:31
@DeeJayLSP DeeJayLSP force-pushed the better-modern-subsection-color branch from af74825 to d433118 Compare November 5, 2025 16:32
@DeeJayLSP DeeJayLSP changed the title Fix prop_subsection_color not scaling with luminance Fix prop_subsection_stylebox color not scaling with luminance Nov 5, 2025
@DeeJayLSP DeeJayLSP force-pushed the better-modern-subsection-color branch from d433118 to 2856935 Compare November 5, 2025 16:38
@Mickeon
Copy link
Member

Mickeon commented Nov 5, 2025

Does this PR affect the look substantially when the "Classic" theme is used?

@DeeJayLSP DeeJayLSP force-pushed the better-modern-subsection-color branch from 2856935 to bbfd2be Compare November 5, 2025 17:02
@DeeJayLSP
Copy link
Contributor Author

Does this PR affect the look substantially when the "Classic" theme is used?

Oops, I should have mentioned this doesn't touch the Classic theme.

@DeeJayLSP DeeJayLSP changed the title Fix prop_subsection_stylebox color not scaling with luminance Modern style: Fix prop_subsection_stylebox color not scaling with luminance Nov 5, 2025
@Repiteo Repiteo merged commit 00af6f7 into godotengine:master Nov 5, 2025
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Nov 5, 2025

Thanks!

@DeeJayLSP DeeJayLSP deleted the better-modern-subsection-color branch November 5, 2025 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants