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

Limit star expansion when measuring/arranging at constrained sizes #17880

Merged
merged 2 commits into from
Oct 16, 2023

Conversation

hartez
Copy link
Contributor

@hartez hartez commented Oct 7, 2023

Description of Change

For constrained measurement, when definition sizes are determined during the measure pass the star sizes are limited by the number of stars which can fit into the constraint. During arrange, if the actual arrange size is different, then the size of a star definition needs to be limited by the number of stars which can fit into the arranged size.

Proposing this as an alternative to #17199 - this is a less extensive change to achieve the same effect.

Issues Fixed

Fixes #17125

@jsuarezruiz jsuarezruiz added the area-layout StackLayout, GridLayout, ContentView, AbsoluteLayout, FlexLayout, ContentPresenter label Oct 9, 2023
@samhouts samhouts added this to the .NET 8 SR1 milestone Oct 9, 2023
@hartez hartez marked this pull request as ready for review October 9, 2023 21:04
@hartez hartez requested a review from a team as a code owner October 9, 2023 21:04
@hartez hartez closed this Oct 10, 2023
@hartez hartez reopened this Oct 10, 2023
@rmarinho rmarinho merged commit 0f764ac into main Oct 16, 2023
47 checks passed
@rmarinho rmarinho deleted the fix-17125 branch October 16, 2023 23:06
@rmarinho rmarinho mentioned this pull request Oct 16, 2023
@PureWeen
Copy link
Member

/backport to release/8.0.1xx-rc2.2

@github-actions
Copy link
Contributor

Started backporting to release/8.0.1xx-rc2.2: https://github.com/dotnet/maui/actions/runs/6632066347

@github-actions github-actions bot locked and limited conversation to collaborators Dec 6, 2023
@samhouts samhouts added the fixed-in-8.0.6 Look for this fix in 8.0.6 SR1! 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-8.0.6 Look for this fix in 8.0.6 SR1!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[regression/8.0.0-rc.1.9171] Grid rows aren't same height on iOS
5 participants