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

[Bug] CarouselView reverting updated CurrentPosition on UWP. #15536

Open
Bloodyairtimer opened this issue Sep 8, 2022 · 3 comments
Open

[Bug] CarouselView reverting updated CurrentPosition on UWP. #15536

Bloodyairtimer opened this issue Sep 8, 2022 · 3 comments
Labels
s/unverified New report that has yet to be verified t/bug 🐛

Comments

@Bloodyairtimer
Copy link

Description

The issue occurs when I use the Position property on the CarouselView to update the carousel to go to the next item.
On UWP the Position is updated via the viewmodel property and the PositionChanged event is fired with the correct position parameter. Right after that the PositionChanged event is fired directly again, this time reverting the previous change.

The callstack is showing the first update that it is triggered by the update from the CurrentPosition property on the viewmodel. The second time there is no callstack other than the event.

Tried to change the binding to oneway. No difference. Sometimes when fast clicking the button on the screen multiple times the screen is navigating to the next view.

Steps to Reproduce

  1. Start the app
  2. Click the next button
  3. See the output window where debug logging is added to show the behavior.

Expected Behavior

When updating the position property on the viewmodel the control should react and navigate to the next page.

Actual Behavior

The screen is bouncing a bit and reverting the position change.

Basic Information

  • Version with issue: XF 5.0.0.2196
  • Last known good version: Unknown.
  • Platform Target Frameworks:
    • UWP: Windows 11 22000

Environment

Reproduction Link

App6.zip

@Bloodyairtimer Bloodyairtimer added s/unverified New report that has yet to be verified t/bug 🐛 labels Sep 8, 2022
@iancona
Copy link

iancona commented Sep 9, 2022

Same issue, not happening on Android with same code. I've also disabled swipegesture on UWP because the scrolling was a mess

@iancona
Copy link

iancona commented Sep 9, 2022

Tried making CarouselView.IsScrollAnimated false as workaround and it seems to work

@Bloodyairtimer
Copy link
Author

That seems to work indeed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
s/unverified New report that has yet to be verified t/bug 🐛
Projects
None yet
Development

No branches or pull requests

3 participants
@iancona @Bloodyairtimer and others