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

[Bug] CarouselView.CurrentItem not working at all #13122

Closed
viktorszekeress opened this issue Dec 12, 2020 · 3 comments · Fixed by #13182
Closed

[Bug] CarouselView.CurrentItem not working at all #13122

viktorszekeress opened this issue Dec 12, 2020 · 3 comments · Fixed by #13182
Assignees
Labels
5.0.0 Regression on 5.0.0 a/carouselview blocker Issue blocks next stable release. Prioritize fixing and reviewing this issue. i/regression t/bug 🐛
Milestone

Comments

@viktorszekeress
Copy link

Description

I have a CarouselView with ItemsSource and CurrentItem bound to dependency properties defined in the page. Then I have 2 buttons to increment and decrement current item. The problem is that setting CarouselView.CurrentItem has no effect on 5.0-pre5. Although it does work properly on version 4.8.

Interesting part is that after swiping the item, current item 'auto-corrects' and starts to work properly.

Steps to Reproduce

  1. Open and run the attached solution
  2. Click Next button
  3. See that CurrentItem item defined in the page is changed, but CarouselView doesn't navigate to the item

Expected Behavior

CarouselView should navigate to CurrentItem.

Actual Behavior

CarouselView stays at initial item.

Basic Information

  • Version with issue: 5.0-pre5
  • Last known good version: 4.8
  • Platform Target Frameworks:
    • iOS: not tested
    • Android: 10.0
    • UWP: not tested

Screenshots

Xamarin Forms 5.0-pre5
5 0-pre5

Xamarin Forms 4.8
4 8

image

image

Reproduction Link

CarouselView.CurrentItem.zip

@viktorszekeress viktorszekeress added s/unverified New report that has yet to be verified t/bug 🐛 labels Dec 12, 2020
@samhouts samhouts added this to New in Triage Dec 12, 2020
@rmarinho rmarinho added 5.0.0 Regression on 5.0.0 i/regression and removed s/unverified New report that has yet to be verified labels Dec 16, 2020
@rmarinho rmarinho moved this from New to Ready For Work in Triage Dec 16, 2020
@rmarinho rmarinho added this to Backlog in CarouselView via automation Dec 16, 2020
@rmarinho rmarinho added this to To do in vNext+1 (5.0.0) via automation Dec 16, 2020
@rmarinho rmarinho moved this from To do to In Progress in vNext+1 (5.0.0) Dec 16, 2020
@rmarinho rmarinho self-assigned this Dec 16, 2020
@rmarinho rmarinho added this to the 5.0.0 milestone Dec 16, 2020
@rmarinho rmarinho added the blocker Issue blocks next stable release. Prioritize fixing and reviewing this issue. label Dec 16, 2020
@rmarinho rmarinho moved this from In Progress to Blockers in vNext+1 (5.0.0) Dec 16, 2020
@rmarinho rmarinho moved this from Backlog to In Progress in CarouselView Dec 17, 2020
@rmarinho rmarinho moved this from Blockers to In Progress in vNext+1 (5.0.0) Dec 17, 2020
rmarinho added a commit that referenced this issue Dec 17, 2020
Triage automation moved this from Ready For Work to Closed Dec 18, 2020
CarouselView automation moved this from In Progress to Done Dec 18, 2020
vNext+1 (5.0.0) automation moved this from In Progress to Done Dec 18, 2020
@johnsheatjx
Copy link

This seems to still be an issue on iOS.

@satzmuthu
Copy link

This seems to still be an issue on iOS.

Yes facing issue in iOS

@jrasbr
Copy link

jrasbr commented Jul 11, 2021

This still does not work on Android. If i set current item to 3 - it will not update the carousel position, but if i set it to 1 or 2 it works fine.
In other words, it will update if CurrentItem is only two places from current position

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
5.0.0 Regression on 5.0.0 a/carouselview blocker Issue blocks next stable release. Prioritize fixing and reviewing this issue. i/regression t/bug 🐛
Projects
CarouselView
  
Done
Triage
  
Closed
Development

Successfully merging a pull request may close this issue.

6 participants