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
Split container hidden sizing #3348
Split container hidden sizing #3348
Conversation
These checks are only required if the widget does not have container. If a parent container is present, the container checks for child visibility when working out it's MinSize.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It does make sense actually, and there is tests \o/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is great, but it should be based on the develop
branch for it to be merged in for next release.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work. Looks good to me :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks
Description:
Addresses Split Container not respecting item's visible status if item is not in a container. When calculating the size of each split, my fix checks if the item is visible and returns a size of 0 if not.
Fixes #3232
Checklist: