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

[Bug] Snap Points not working in CollectionView with GridItemsLayout #11938

Closed
missy-bowles opened this issue Aug 26, 2020 · 2 comments · Fixed by #11941
Closed

[Bug] Snap Points not working in CollectionView with GridItemsLayout #11938

missy-bowles opened this issue Aug 26, 2020 · 2 comments · Fixed by #11941
Assignees
Labels
a/collectionview e/2 🕑 2 in-progress This issue has an associated pull request that may resolve it! p/Android t/bug 🐛
Milestone

Comments

@missy-bowles
Copy link

Description

A CollectionView that uses a GridItemsLayout with SnapPointsType="Mandatory" does not have snap points and scrolling stops mid column on Android. iOS works as expected.

Expected Behavior

CollectionView should use snap points

Actual Behavior

CollectionView does not have snap points

Basic Information

  • Version with issue: Xamarin.Forms 4.6.0
  • Last known good version: Unknown
  • IDE: VS for Mac 8.7.3
  • Platform Target Frameworks:
    • Android: 9.0
  • Android Support Library Version:
  • Nuget Packages:
  • Affected Devices: Android

Screenshots

Reproduction Link

SnapPointsRepro.zip

Workaround

None that I know of

@missy-bowles missy-bowles added s/unverified New report that has yet to be verified t/bug 🐛 labels Aug 26, 2020
@jsuarezruiz jsuarezruiz added a/collectionview p/Android and removed s/unverified New report that has yet to be verified labels Aug 27, 2020
@jsuarezruiz
Copy link
Contributor

Verified. Happens with Xamarin.Forms 4.6, 4.7 and 4.8. Dynamically updating the SnapPointsType value works fine.

@jsuarezruiz jsuarezruiz self-assigned this Aug 27, 2020
@samhouts samhouts added the in-progress This issue has an associated pull request that may resolve it! label Aug 27, 2020
@samhouts samhouts added this to the 5.0.0 milestone Aug 31, 2020
@InquisitorJax
Copy link

@jsuarezruiz "Dynamically updating the SnapPointsType value works fine." - is that a work-around to the issue? If so, what does that mean (how to)?

rmarinho added a commit that referenced this issue Oct 29, 2020
…out (#11941) fixes #11938

* Added repro sample

* Fixed the issue

* Added comment

* Fixed build error

* [Controls ] Fix merge

Co-authored-by: Rui Marinho <me@ruimarinho.net>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a/collectionview e/2 🕑 2 in-progress This issue has an associated pull request that may resolve it! p/Android t/bug 🐛
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants