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

[Bug] RefreshView with CollectionView child cannot contain GridItemsLayout with Span greater than 1 in combination with EmptyView on UWP #11794

Closed
tomreich opened this issue Aug 15, 2020 · 0 comments · Fixed by #11913
Assignees
Labels
a/collectionview a/refreshview 🔄 e/3 🕒 3 in-progress This issue has an associated pull request that may resolve it! p/UWP t/bug 🐛
Milestone

Comments

@tomreich
Copy link

Description

Creating a RefreshView that has a CollectionView that has both CollectionView.ItemsLayout set vertical with a span of greater than one AND CollectionView.EmptyView causes a crash in the UWP environment. No crash in Android. Haven't tried in iOS.

Steps to Reproduce

The following XAML produces the error when the CollectionView is wired up to an Observable Collection. Comment out either CollectionView.ItemsLayout or CollectionView.EmptyView and the issue goes away.

<RefreshView> <CollectionView x:Name="testCollectionView"> <CollectionView.ItemsLayout> <GridItemsLayout Orientation="Vertical" Span="3" /> </CollectionView.ItemsLayout> <CollectionView.ItemTemplate> <DataTemplate> <Frame> <Label Text="{Binding .}" /> </Frame> </DataTemplate> </CollectionView.ItemTemplate> <CollectionView.EmptyView> <StackLayout IsVisible="{Binding IsEmpty}"> <Label Text="Welcome to crashtown" /> </StackLayout> </CollectionView.EmptyView> </CollectionView> </RefreshView>

Expected Behavior

CollectionView Renders successfully

Actual Behavior

Application crash

Basic Information

  • Version with issue: Xamarin.Forms 4.8.0.1269
  • Last known good version:
  • IDE: Visual Studio 2019
  • Platform Target Frameworks:
    • iOS: N/A
    • Android: API 28
    • UWP: 18362
  • Android Support Library Version: N/A
  • Nuget Packages: Xamarin.Forms 4.8.0.1269
  • Affected Devices:

Screenshots

N/A

Reproduction Link

N/A

Workaround

N/A

@tomreich tomreich added s/unverified New report that has yet to be verified t/bug 🐛 labels Aug 15, 2020
@jsuarezruiz jsuarezruiz removed the s/unverified New report that has yet to be verified label Aug 25, 2020
@jsuarezruiz jsuarezruiz self-assigned this Aug 25, 2020
@samhouts samhouts added the in-progress This issue has an associated pull request that may resolve it! label Aug 25, 2020
@samhouts samhouts added this to the 5.0.0 milestone Aug 25, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a/collectionview a/refreshview 🔄 e/3 🕒 3 in-progress This issue has an associated pull request that may resolve it! p/UWP t/bug 🐛
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants