-
-
Notifications
You must be signed in to change notification settings - Fork 21.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TextEdit: Correct typo that broke custom selected font color #30018
TextEdit: Correct typo that broke custom selected font color #30018
Conversation
Try to use the word fix/fixes/close/closes/resolve/resolves before a issue number so it's tagged automatically for closing when the commit is merged. |
Looks good, but there are some more occurrences of trying to set
I would suggest to also rename the variables used internally like |
I decided not to change text_selected_color because that really is the name of the setting. |
Ideally yes, but that would break compatibility with themes made with earlier Godot version, so it can't be done now. We can rename it in 4.0 though, I'll add a mention to it in #16863. Changes look good, could you squash commits together? |
Change several font_selected_color to font_color_selected; the actual name of the override
Thanks, and congrats for your first Godot PR :) |
Cherry-picked for 3.1.2. |
text_edit's cache was trying to get "font_selected_color" but the colors name as defined in default_theme.cpp is "font_color_selected"
Fixes #27760.