Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Vertical scrolling not working for CarouselView and CustomLayouts #7814

Open
ShariatPanah opened this issue Jun 7, 2022 · 13 comments
Open
Labels
area-controls-collectionview CollectionView, CarouselView, IndicatorView platform/android 🤖 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Milestone

Comments

@ShariatPanah
Copy link

ShariatPanah commented Jun 7, 2022

Description

so when I have a vertical scrollview on my page that:
1. has a carouselview as one of its children at the top (I assume this happens to CustomLayouts too)
2. and at the bottom there is another scrollview with horizontal scrolling
I'm facing a problem, when i scroll down by touching the carouselview, it works fine, but when i iterate over the bottom scrollview (horizontally) and want to get back up there by scrolling vertically using the CarouselView it doesn't perform the scrolling until I touch the bottom scrollview and going a little bit vertically, then i can continue scrolling up by touching the carouselview.

NOTE: as I said before I think this happens to CustomLayouts too, please check that yourself or let me know to create a sample for it also

I'm attaching a screen recording and the sample project below

and by the way, the backgroundcolor of frame or even the label in the carouselview datatemplate doesn't show the color that is binding to, maybe I'm doing something wrong or is it a bug?

Here is the screen recording:

CarouselAndCustomLayoutBug.mp4

Steps to Reproduce

and here is the sample project:
CustomLayoutBug.zip

Version with bug

6.0 Release Candidate 3

Last version that worked well

Unknown/Other

Affected platforms

Android

Affected platform versions

Android 10.0

Did you find any workaround?

Not yet

Relevant log output

No response

@ShariatPanah ShariatPanah added s/needs-verification Indicates that this issue needs initial verification before further triage will happen t/bug Something isn't working labels Jun 7, 2022
@jsuarezruiz jsuarezruiz added the area-controls-collectionview CollectionView, CarouselView, IndicatorView label Jun 7, 2022
@AswinPG
Copy link

AswinPG commented Jun 8, 2022

I think these could be related
#6412
xamarin/XamarinCommunityToolkit#591
@ShariatPanah Can you check if it's the swipe angle that's causing the issue..

@ShariatPanah
Copy link
Author

@AswinPG As I showed in the video, I'm performing an exact vertical angle to swipe, and I dont think it's similar to your issue, maybe at framework level yes

@VincentBu
Copy link

not repro with vs main build 32609.239.

@VincentBu VincentBu added s/verified Verified / Reproducible Issue ready for Engineering Triage and removed s/needs-verification Indicates that this issue needs initial verification before further triage will happen labels Jun 10, 2022
@ShariatPanah
Copy link
Author

were you able to repro this issue? i ask because you said "NOT REPRO" but you added verified tag to this issue
I'm confused :|

@ShariatPanah
Copy link
Author

any update on this @jsuarezruiz ?

@samhouts samhouts removed the s/verified Verified / Reproducible Issue ready for Engineering Triage label Apr 5, 2023
@homeyf homeyf added the s/triaged Issue has been reviewed label May 9, 2023
@homeyf
Copy link

homeyf commented May 9, 2023

Verified this issue with Visual Studio Enterprise 17.6.0 Preview 5.0. Can repro on Android platform with above project.
repro project: CustomLayoutBug.zip
demo0004

@homeyf homeyf added the s/verified Verified / Reproducible Issue ready for Engineering Triage label May 9, 2023
@SabigGasim
Copy link

This is still happening in Visual Studio 17.7 , dontet version 7.0.400

@ShariatPanah
Copy link
Author

guys it's been almost 16 months from when i reported this issue!!!! what is really going on? if you cant fix this issue just say it, if i want to fix this just tell me where the problem is? i may be able to fix it :/

@Jun-FH
Copy link

Jun-FH commented Sep 12, 2023

@ShariatPanah
Have you tried with this:
<Carousel {...} panGestureHandlerProps={{ activeOffsetX: [-10, 10], }} />

Reference: dohooo/react-native-reanimated-carousel#125

@ShariatPanah
Copy link
Author

@Jun-FH we're on .net maui, this syntax is not a valid one in maui, but thanks for your help

@ShariatPanah
Copy link
Author

which version? i tested with 8.0.40 and the issue is still there

@dotnet-policy-service dotnet-policy-service bot removed the s/try-latest-version Please try to reproduce the potential issue on the latest public version label Jun 6, 2024
@ShariatPanah
Copy link
Author

also tested with nightly build version 8.0.60-ci.net8.24305.3 and still i'm facing the issue

@jsuarezruiz jsuarezruiz added this to the Backlog milestone Jun 7, 2024
@RhomGit
Copy link

RhomGit commented Jun 23, 2024

I hooked up the OnCarouselViewScrolled event as described here: https://learn.microsoft.com/en-us/dotnet/maui/user-interface/controls/carouselview/scrolling?view=net-maui-8.0
The OnCarouselViewScrolled event doesn't ever seem to fire when vertically scrolling so yes, I am experiencing this issue also.

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-controls-collectionview CollectionView, CarouselView, IndicatorView platform/android 🤖 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

9 participants