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

Add disabled theme icons for CheckBox #37755

Merged
merged 1 commit into from
Mar 23, 2021
Merged

Conversation

KoBeWi
Copy link
Member

@KoBeWi KoBeWi commented Apr 10, 2020

Resolves #8640
image

HOWEVER I have no idea how to optimize the new images, so if anyone could advise what to use it would be nice >_>

@Calinou
Copy link
Member

Calinou commented Apr 10, 2020

HOWEVER I have no idea how to optimize the new images, so if anyone could advise what to use it would be nice >_>

I use oxipng (typically) or sometimes OptiPNG with the following options:

# For small images. Zopfli optimization (`-Z`) results in better compression.
oxipng -Zso6 image.png
# or:
optipng -o6 -zm1-9 -strip all image.png

# For large images. Zopfli is slow to compress on large images, so it'd take too long to be usable.
oxipng -so6 image.png
# or:
optipng -o6 -strip all image.png

We could add Imgbot to the Godot repositories to perform this automatically.

@KoBeWi
Copy link
Member Author

KoBeWi commented Apr 10, 2020

Ok, images optimized.

@Calinou Calinou added this to the 4.0 milestone Jan 5, 2021
Copy link
Member

@Calinou Calinou left a comment

Choose a reason for hiding this comment

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

The default project theme will likely be overhauled for 4.0, but this is a good change in the meantime that can be cherry-picked to the 3.x branch after 3.3 is released.

@Calinou Calinou added the cherrypick:3.x Considered for cherry-picking into a future 3.x release label Mar 20, 2021
@akien-mga akien-mga merged commit 036a50e into godotengine:master Mar 23, 2021
@akien-mga
Copy link
Member

Thanks!

@KoBeWi KoBeWi deleted the check_down branch March 23, 2021 12:06
@akien-mga
Copy link
Member

Cherry-picked for 3.4.

@akien-mga akien-mga removed the cherrypick:3.x Considered for cherry-picking into a future 3.x release label Apr 26, 2021
@Deozaan
Copy link

Deozaan commented Nov 7, 2021

I just updated my project from 3.3.4 to 3.4 and this broke things for me. I'm using a custom theme based off the Editor theme, and it seems the Editor theme was not updated for 3.4 to include the new disabled icons.

So now I have mismatched icons when disabling/enabling any CheckBox nodes.

godot-3 4-checkbox

Godot 3 4 disabled CheckBox

Because the Editor theme doesn't have the disabled icons, I can't just import them into my theme.

Godot 3 4 Editor Theme CheckBox Icons Import

Meanwhile, the default theme of course does have them.

Godot 3 4 Default Theme CheckBox Icons Import

@Zireael07
Copy link
Contributor

@Deozaan: There were enough changes in EditorTheme that your best bet is probably to reexport the theme anew.

@KoBeWi
Copy link
Member Author

KoBeWi commented Nov 7, 2021

@Deozaan You can find these icons under EditorIcons group in the editor theme. They are called GUIChecked/UncheckedDisabled.

@Deozaan
Copy link

Deozaan commented Nov 8, 2021

There were enough changes in EditorTheme that your best bet is probably to reexport the theme anew.

@Zireael07 That's probably a good idea, but I don't see how that will help with the missing icons.

You can find these icons under EditorIcons group in the editor theme. They are called GUIChecked/UncheckedDisabled.

@KoBeWi I don't see them.

Godot 3 4 Editor Theme EditorIcons Disabled

@KoBeWi
Copy link
Member Author

KoBeWi commented Nov 8, 2021

You are right, they only exist in 4.0. They were added in #51722 and never backported. I opened an issue: #54742

In the meantime, if you need to use these icons, you can get them here:
https://github.com/godotengine/godot/blob/master/editor/icons/GuiRadioCheckedDisabled.svg
https://github.com/godotengine/godot/blob/master/editor/icons/GuiRadioUncheckedDisabled.svg
https://github.com/godotengine/godot/blob/master/editor/icons/GuiCheckedDisabled.svg
https://github.com/godotengine/godot/blob/master/editor/icons/GuiUncheckedDisabled.svg

@Deozaan
Copy link

Deozaan commented Nov 8, 2021

Thank you!

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.

CheckBox and CheckButton don't change appearance when set to disabled.
5 participants