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

[Bug] [iOS] CollectionView - blank spaces beetween items on updating list when items has different heights #11511

Closed
mtopolewski opened this issue Jul 22, 2020 · 8 comments

Comments

@mtopolewski
Copy link

mtopolewski commented Jul 22, 2020

There some blank spaces appears between items on CollectionView on iOS if items has different heights. Best way to show these spaces is updating the list, for example inserting new item at the top. This problem occurs only on iOS, works fine on Android.

Steps to Reproduce

  1. Use my reproduction repo at the bottom of this post
  2. Run application on iOS device/simulator
  3. Click on "Get better stuff" button

Expected Behavior

These blank spaces should not appears after inserting new item to list, like on image below:
Simulator Screen Shot - iPhone 11 - 2020-07-22 at 16 47 44

Actual Behavior

Blank spaces appears after inserting new item to list. Look how it looks on image below:

Simulator Screen Shot - iPhone 11 - 2020-07-22 at 16 47 41

Additional comments

Besides, CollectionView doesn't scroll smoothly, when you try to implement pagination on your list and if items on this list have different heights you can observe that list is jumping a bit, it should stay on same scrolling position. Look at video below:
https://streamable.com/b4tsmv

Basic Information

  • Xamarin Forms: 4.7.0.11.42
  • Visual Studio for Mac 8.6.5 (build 23)
  • Xcode ver 11.5

Reproduction Link

https://github.com/mariaszek9003/XamarinFormsCollectionViewTest

@mtopolewski mtopolewski added s/unverified New report that has yet to be verified t/bug 🐛 labels Jul 22, 2020
@samhouts samhouts added this to New in Triage Jul 22, 2020
@joix97
Copy link

joix97 commented Jul 24, 2020

I experience that, too. It's quite annoying bug

@hartez hartez self-assigned this Jul 24, 2020
@hartez hartez removed the s/unverified New report that has yet to be verified label Jul 24, 2020
@hartez hartez added this to Backlog in CollectionView via automation Jul 24, 2020
@hartez hartez removed this from New in Triage Jul 24, 2020
@hartez hartez added the e/5 🕔 5 label Jul 24, 2020
@DuncWatts
Copy link

In the example you haven't got ItemSizingStrategy="MeasureAllItems" set, the default behaviour is to measure the first item and use that size for all items.

@mtopolewski
Copy link
Author

mtopolewski commented Jul 29, 2020

@DuncWatts MeasureAllItems is a default value for ItemSizingStrategy, so it not make any difference if I set it explicitly https://docs.microsoft.com/pl-pl/dotnet/api/xamarin.forms.itemsizingstrategy?view=xamarin-forms

@samhouts samhouts added this to the 5.0.0 milestone Aug 13, 2020
@samhouts samhouts added this to To do in vNext+1 (5.0.0) Aug 13, 2020
@Tommigun1980
Copy link

Maybe this is the root cause of #12554 and #12555?

If so Xamarin please fix this as it introduces so many follow-up bugs that it's almost impossible to use the CollectionView. It would be such a useful component.

Thanks.

@Codelisk

This comment has been minimized.

@joix97

This comment has been minimized.

@Tommigun1980

This comment has been minimized.

@hartez
Copy link
Contributor

hartez commented Feb 8, 2021

This is fixed in 5.0.0.1931.

@hartez hartez closed this as completed Feb 8, 2021
CollectionView automation moved this from Review Backlog to Done Feb 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Development

No branches or pull requests

7 participants