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

[Bug] CarouselView Loop=True default freezes iOS app #12574

Closed
rachelkang opened this issue Oct 22, 2020 · 0 comments · Fixed by #12787
Closed

[Bug] CarouselView Loop=True default freezes iOS app #12574

rachelkang opened this issue Oct 22, 2020 · 0 comments · Fixed by #12787
Assignees
@rmarinho
Labels
a/carouselview blocker Issue blocks next stable release. Prioritize fixing and reviewing this issue. p/iOS 🍎 t/bug 🐛
Projects
vNext+1 (5.0.0)
Milestone
5.0.0

Comments

@rachelkang
Copy link
Contributor

Description

In Xamarin.Forms 5.0, the app freezes on iOS when you swipe through the CarouselView with Loop=True (or default, not specified).

Steps to Reproduce

  1. Run the attached repro app
  2. Swipe through the CarouselView on the first tab (Loop=False)
    Notice that it works as expected
  3. Switch to the second tab (Loop=True)
  4. Swipe through the CarouselView on the second tab
    Notice that the app freezes mid-swipe
    Not only does the CarouselView freeze, but the entire app freezes (i.e. try switching back to the first tab)

Expected Behavior

CarouselView should not freeze and looping should be functional

Actual Behavior

CarouselView freezes entire app while it is being swiped through (unless Loop=false)

Basic Information

  • Version with issue: 5.0.0.1558-pre3
  • Last known good version: X
  • IDE: VSMac
  • Platform Target Frameworks:
    • iOS: 14.2
  • Affected Devices: iOS

Reproduction Link

CarouselViewLoopTest.zip

Workaround

Loop=False prevents freezing, but no workaround found for enabling looping
@rachelkang rachelkang added t/bug 🐛 p/iOS 🍎 a/carouselview s/unverified New report that has yet to be verified labels Oct 22, 2020
@rachelkang rachelkang added this to To do in vNext+1 (5.0.0) via automation Oct 22, 2020
@rmarinho rmarinho moved this from To do to Blockers in vNext+1 (5.0.0) Oct 29, 2020
@rmarinho rmarinho added blocker Issue blocks next stable release. Prioritize fixing and reviewing this issue. and removed s/unverified New report that has yet to be verified labels Oct 29, 2020
@PureWeen PureWeen added this to the 5.0.0 milestone Nov 5, 2020
rmarinho added a commit that referenced this issue Nov 9, 2020
@rmarinho
[Controls] Add repo for issue #12574
5ac4f7b
@rmarinho rmarinho moved this from Blockers to In Progress in vNext+1 (5.0.0) Nov 9, 2020
@rmarinho rmarinho mentioned this issue Nov 10, 2020
Merged
2 tasks
vNext+1 (5.0.0) automation moved this from In Progress to Done Nov 12, 2020
rmarinho added a commit that referenced this issue Nov 12, 2020
@rmarinho @jsuarezruiz
[iOS] Remove hardcoded value on CarouselView loop calculations (#12787)
06ccda8 
fixes #12574 fixes #12787

* [Controls] Add repo for issue #12574

* [iOS] Handle loop calculations when ItemsSource count is small

* [Controls] Update UITest with remove item test

Co-authored-by: Javier Suárez <javiersuarezruiz@hotmail.com>
@rmarinho rmarinho mentioned this issue Nov 13, 2020
Merged
2 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@rmarinho rmarinho

Labels
a/carouselview blocker Issue blocks next stable release. Prioritize fixing and reviewing this issue. p/iOS 🍎 t/bug 🐛
Projects
No open projects
vNext+1 (5.0.0)
Done
Milestone
5.0.0
Development

Successfully merging a pull request may close this issue.

[iOS] Remove hardcoded value on CarouselView loop calculations xamarin/Xamarin.Forms
3 participants
@rmarinho @PureWeen @rachelkang