Skip to content
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

Fix Illegible text in audio bus editor #50593

Merged
merged 1 commit into from
Jul 20, 2021
Merged

Fix Illegible text in audio bus editor #50593

merged 1 commit into from
Jul 20, 2021

Conversation

SirQuartz
Copy link
Contributor

@SirQuartz SirQuartz commented Jul 18, 2021

This pull request fixes an issue with the audio bus editor where the text would be illegible regardless of the theme being used.

Resolves: #50583

@SirQuartz SirQuartz requested a review from a team as a code owner July 18, 2021 20:18
@Calinou Calinou added bug topic:editor usability cherrypick:3.x Considered for cherry-picking into a future 3.x release labels Jul 18, 2021
@Calinou Calinou added this to the 4.0 milestone Jul 18, 2021
@YuriSizov
Copy link
Contributor

YuriSizov commented Jul 18, 2021

I'm not sure why you would need to change anything in the default theme. You just need to change the related color in the editor theme based on the dark theme flag. If there is no color constant specific to the bus name for some reason, it needs to be defined.

This also isn't the greatest color in general, midtones are still not very readable, on either theme preset. And the empty line is unnecessary.

@YuriSizov
Copy link
Contributor

YuriSizov commented Jul 18, 2021

@SirQuartz That just means that there are definitions in the editor theme missing. Those definitions are only created in the editor_themes.cpp file. The default_theme.cpp files should not be touched to fix an issue with the editor theme, because it does not contain the editor theme. It contains the default engine theme, the one you see in the projects, not in the editor.

If you think that every LineEdit in the editor, provided they are set to not be editable, is affected by this you need to create or adjust definitions for font_uneditable_color in the editor theme definitions. Somewhere around here:

https://github.com/godotengine/godot/blob/e3a86d54c3be39982384a5e89c151ecfefc62b00/editor/editor_themes.cpp#L1022-L1032

There is likely already a good color available that respects the color scheme that you can use here.

Alternatively we can consider swapping the LineEdit for a Label when it's not editable and using some other color.

This pull request fixes an issue with the audio bus editor where the text would be illegible regardless of the theme being used.
@SirQuartz
Copy link
Contributor Author

SirQuartz commented Jul 18, 2021

@pycbouh Ok, I've fixed everything, and this should be a much cleaner implementation.

default_theme (2)
light_theme (2)

@akien-mga akien-mga merged commit d7811c7 into godotengine:master Jul 20, 2021
@akien-mga
Copy link
Member

Thanks!

@akien-mga
Copy link
Member

It seems like font_uneditable_color doesn't exist in 3.x so a different fix would be needed there.

@akien-mga akien-mga removed cherrypick:3.3 cherrypick:3.x Considered for cherry-picking into a future 3.x release labels Jul 20, 2021
@SirQuartz
Copy link
Contributor Author

SirQuartz commented Jul 20, 2021

It seems like font_uneditable_color doesn't exist in 3.x so a different fix would be needed there.

I see, I'll look into a solution for 3.x as well.

@SirQuartz
Copy link
Contributor Author

Thanks!

@akien-mga See: #50646

@SirQuartz SirQuartz deleted the patch-24 branch July 22, 2021 18:44
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.

Illegible texts in audio bus editor when using a light theme
5 participants