Skip to content

Prevent measure in flexlayout during arrange pass#22411

Merged
PureWeen merged 10 commits into
mainfrom
foda/FlexLayoutCrash
May 19, 2024
Merged

Prevent measure in flexlayout during arrange pass#22411
PureWeen merged 10 commits into
mainfrom
foda/FlexLayoutCrash

Conversation

@Foda
Copy link
Copy Markdown
Contributor

@Foda Foda commented May 14, 2024

Description of Change

Prevent measure in flexlayout during arrange pass, which fixes a layout cycle crash on Windows. No automated test for this one as I haven't quite figured out how to replicate a 125% display res change on CI

Issues Fixed

Fixes #22209 #9075

@Foda Foda added platform/windows area-layout StackLayout, GridLayout, ContentView, AbsoluteLayout, FlexLayout, ContentPresenter layout-flex FlexLayout issues labels May 14, 2024
@Foda Foda requested a review from PureWeen May 14, 2024 21:49
@Foda Foda requested a review from a team as a code owner May 14, 2024 21:49
@Foda Foda requested a review from StephaneDelcroix May 14, 2024 21:49
@MartyIX
Copy link
Copy Markdown
Contributor

MartyIX commented May 15, 2024

@Foda
Copy link
Copy Markdown
Contributor Author

Foda commented May 15, 2024

Are these two comments:

possibly relevant here?

Yup! This PR also fixes that issue

@PureWeen
Copy link
Copy Markdown
Member

/rebase

@github-actions github-actions Bot force-pushed the foda/FlexLayoutCrash branch from 4e97369 to bc36d3a Compare May 16, 2024 20:40
PureWeen
PureWeen previously approved these changes May 16, 2024
PureWeen
PureWeen previously approved these changes May 17, 2024
@PureWeen PureWeen merged commit 60a46c5 into main May 19, 2024
@PureWeen PureWeen deleted the foda/FlexLayoutCrash branch May 19, 2024 13:52
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 19, 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.60 fixed-in-9.0.0-preview.5.24307.10 layout-flex FlexLayout issues platform/windows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Windows] FlexLayout crashes when using a custom display scale factor value (regression)

5 participants