Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

Comment out measure all strategy on iOS 15 for grouping #15355

Merged
merged 3 commits into from
May 18, 2022
Merged

Conversation

jfversluis
Copy link
Member

Description of Change

Porting (potential) fix from .NET MAUI to Forms. Also see: dotnet/maui#6878

Issues Resolved

API Changes

None

Platforms Affected

  • iOS

Behavioral/Visual Changes

None

Before/After Screenshots

Not applicable

Testing Procedure

PR Checklist

  • Targets the correct branch
  • Tests are passing (or failures are unrelated)

@softlion
Copy link
Contributor

softlion commented May 6, 2022

I'm not using the MeasureAll mode already to "fix" this issue.

It will fix the crash but the cell still won't auto resize. I know that's by design on iOS as to resize a cell you must replace it (ie: delete and insert in the same transation).

@jfversluis
Copy link
Member Author

jfversluis commented May 6, 2022

Sounds like you've got a better method in mind, I'll be happy to hear it or see a PR :)

Although I think this only skips the invalidate for header cells, right?

@jfversluis jfversluis merged commit 6cc1cfd into 5.0.0 May 18, 2022
@jfversluis jfversluis deleted the fix-15230 branch May 18, 2022 18:38
rotorgames pushed a commit to rotorgames/Xamarin.Forms that referenced this pull request Jun 22, 2022
* Update ItemsViewLayout.cs

* Update ItemsViewLayout.cs

* Make check for iOS 15.4 and higher
# Conflicts:
#	Xamarin.Forms.Platform.iOS/Forms.cs
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] iOS 15.4 Crash on CollectionView with Grouping
3 participants