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

Make sure that Star rows/columns get an auto-sizing measure when constraints are infinite #1883

Merged

Conversation

hartez
Copy link
Contributor

@hartez hartez commented Jul 30, 2021

When given an infinite constraint along an axis with a Star value, GridLayout can't distribute the Star values normally (because they'd all be infinite).

Instead, GridLayout treats Star values in this case as Auto values, and gives the elements inside the values they request.

In some circumstances, the GridLayout was doing this without ever actually measuring the element, giving the Star value the default of zero. These changes ensure that the element is actually measured in these circumstances.

@hartez hartez changed the base branch from main to release/6.0.1xx-preview7 July 30, 2021 20:25
@rachelkang rachelkang self-requested a review July 30, 2021 21:22
Copy link
Member

@rachelkang rachelkang left a comment

Choose a reason for hiding this comment

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

LGTM once checks all turn green!

@PureWeen PureWeen merged commit a033552 into release/6.0.1xx-preview7 Jul 31, 2021
@PureWeen PureWeen deleted the fix-star-sizing-with-infinite-constraints branch July 31, 2021 18:13
@Redth Redth added this to the 6.0.100-preview.7 milestone Aug 2, 2021
@samhouts samhouts added area-layout StackLayout, GridLayout, ContentView, AbsoluteLayout, FlexLayout, ContentPresenter layout-grid labels Jul 11, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Dec 22, 2023
@samhouts samhouts added the fixed-in-6.0.100-preview.7.3 Look for this fix in 6.0.100-preview.7.3! label Aug 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-layout StackLayout, GridLayout, ContentView, AbsoluteLayout, FlexLayout, ContentPresenter fixed-in-6.0.100-preview.7.3 Look for this fix in 6.0.100-preview.7.3! layout-grid
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants