Skip to content

Comments

fix: address inactive text colour#90

Merged
sgoudham merged 2 commits intocatppuccin:mainfrom
Charlie-Eeles:fix-inactive-text-colour
Jul 17, 2025
Merged

fix: address inactive text colour#90
sgoudham merged 2 commits intocatppuccin:mainfrom
Charlie-Eeles:fix-inactive-text-colour

Conversation

@Charlie-Eeles
Copy link
Contributor

@Charlie-Eeles Charlie-Eeles commented Jun 30, 2025

Fixes #87

override_text_color is overriding all text colours, including inactive values, this means things like input hint text shows the same as actual input.
This PR removes that setting and leaves it to more specific colour setting properties, like fg_stroke.

Examples (note the input hint text of "Enter a todo...")

Mocha before:after

Screenshot 2025-06-30 at 09 43 48
Screenshot 2025-06-30 at 09 45 58

Macchiato before:after

Screenshot 2025-06-30 at 09 43 52
Screenshot 2025-06-30 at 09 46 01

Frappe before:after

Screenshot 2025-06-30 at 09 43 56
Screenshot 2025-06-30 at 09 46 04

Latte before:after

Screenshot 2025-06-30 at 09 43 59
Screenshot 2025-06-30 at 09 46 08

@zanciks
Copy link

zanciks commented Jul 3, 2025

Just now seeing this. Loving the changes posted so far, seems like what you have should already resolve my ticket. However, I should have some time this weekend to get this working on the official egui demo to test it out further, so I will post an update once I do!

@Charlie-Eeles Charlie-Eeles marked this pull request as ready for review July 17, 2025 16:59
Copy link
Contributor

@sgoudham sgoudham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Happy to approve based on the screenshots, thanks!

@sgoudham sgoudham changed the title fix: address inactive text colour (issue #87) fix: address inactive text colour Jul 17, 2025
@sgoudham sgoudham merged commit 9468d77 into catppuccin:main Jul 17, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Text color doesn't fade on inactive fields, inconsistent with default egui themes

3 participants