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 a h_separation between icons in CheckButton/CheckBox #88615

Conversation

Rindbee
Copy link
Contributor

@Rindbee Rindbee commented Feb 21, 2024

Previously, the h_separation between internal elements and custom elements was added when text was not empty. That is, this h_separation does not exist when there is a valid custom icon but text is empty.

Now, the h_separation between the internal element and the custom element is added when the internal element and any custom element exist (both width are greater than 0).

Fix #88505.

scene/gui/check_box.cpp Outdated Show resolved Hide resolved
scene/gui/check_button.cpp Outdated Show resolved Hide resolved
Previously, the `h_separation` between internal elements and custom elements
was added when `text` was not empty. That is, this `h_separation` does not
exist when there is a valid custom `icon` but `text` is empty.

Now, the `h_separation` between the internal element and the custom element
is added when the internal element and any custom element exist (both width
are greater than `0`).
@Rindbee Rindbee force-pushed the add-h_separation-between-icons-in-CheckButton-and-CheckBox branch from f964bf3 to 5de496d Compare February 21, 2024 09:53
@akien-mga akien-mga merged commit 7e567c4 into godotengine:master Feb 26, 2024
16 checks passed
@akien-mga
Copy link
Member

Thanks!

@Rindbee Rindbee deleted the add-h_separation-between-icons-in-CheckButton-and-CheckBox branch February 28, 2024 05:15
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.

Node icon misplaced in Create New Scene dialog
4 participants