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

[iOS] CollectionView item sizing wrong after refresh #20443

Open
naaeef opened this issue Feb 8, 2024 · 4 comments · May be fixed by #21736
Open

[iOS] CollectionView item sizing wrong after refresh #20443

naaeef opened this issue Feb 8, 2024 · 4 comments · May be fixed by #21736
Labels
area-controls-collectionview CollectionView, CarouselView, IndicatorView delighter-sc migration-compatibility Xamarin.Forms to .NET MAUI Migration, Upgrade Assistant, Try-Convert platform/iOS 🍎 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Milestone

Comments

@naaeef
Copy link

naaeef commented Feb 8, 2024

Description

On iOS, after refreshing a CollectionView in a RefreshView, items are not sized correctly. After scrolling down and up, it is displayed correctly.

CollectionView.Error.mov

This issue is currently blocking us from migrating to MAUI.

Steps to Reproduce

  1. Create a new MAUI app
  2. Add a RefreshView and and CollectionView
  3. Use a DataTemplateSelector with two DataTemplates having different heights
  4. Run the app on iOS, refresh the CollectionView

Link to public reproduction project repository

https://github.com/naaeef/CollectionViewReproductionSample

Version with bug

8.0.6

Is this a regression from previous behavior?

Yes, this used to work in Xamarin.Forms

Last version that worked well

Unknown/Other

Affected platforms

iOS

Affected platform versions

No response

Did you find any workaround?

No.

Relevant log output

No response

@naaeef naaeef added the t/bug Something isn't working label Feb 8, 2024
@jsuarezruiz jsuarezruiz added platform/iOS 🍎 area-controls-collectionview CollectionView, CarouselView, IndicatorView labels Feb 8, 2024
@PureWeen PureWeen added the migration-compatibility Xamarin.Forms to .NET MAUI Migration, Upgrade Assistant, Try-Convert label Feb 8, 2024
@PureWeen
Copy link
Member

PureWeen commented Feb 8, 2024

Related #15177

@PureWeen PureWeen added this to the Backlog milestone Feb 8, 2024
@ghost
Copy link

ghost commented Feb 8, 2024

We've added this issue to our backlog, and we will work to address it as time and resources allow. If you have any additional information or questions about this issue, please leave a comment. For additional info about issue management, please read our Triage Process.

@XamlTest XamlTest added s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed labels Mar 19, 2024
@XamlTest
Copy link

Verified this on VS 17.10.0 Preview 2.0(8.0.7 and 8.0.10). Repro on iOS 17.2 with below Project:
TestCollectionView.zip

@softeip softeip linked a pull request Apr 9, 2024 that will close this issue
@samhouts samhouts removed s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed labels Jul 3, 2024
@samhouts samhouts added s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed labels Jul 10, 2024
@rmarinho
Copy link
Member

This seems fixed on net9 using CV2 I added a test here manually for now but seems to work correctly @naaeef do you mind pull my branch and try to see validate is fixed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-controls-collectionview CollectionView, CarouselView, IndicatorView delighter-sc migration-compatibility Xamarin.Forms to .NET MAUI Migration, Upgrade Assistant, Try-Convert platform/iOS 🍎 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants