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

[Bug] SwipeView does not work on Android if child has TapGestureRecognizer #11333

Closed
RedChops opened this issue Jul 6, 2020 · 5 comments
Closed
Assignees
Labels
a/swipeview e/2 🕑 2 in-progress This issue has an associated pull request that may resolve it! p/Android p/iOS 🍎 t/bug 🐛
Milestone

Comments

@RedChops
Copy link

RedChops commented Jul 6, 2020

Description

If the child element of a SwipeView has a TapGestureRecognizer attached (have not tried other gesture recognizers), the element will not swipe. This only affects Android, on iOS the swipe gesture works perfectly okay. This is easiest to see if the child element is a Frame, but the situation is the same regardless of layout type.

Steps to Reproduce

  1. Build attached project
  2. Attempt to swipe on the list elements in both Android and iOS

Expected Behavior

The frame should swipe to the left (SwipeView.RightItems) and expose two actions

Actual Behavior

The frame does not move on Android.

Basic Information

  • Version with issue: 4.6.0.1073 and all later
  • Last known good version: 4.6.0.967
  • IDE:
  • Platform Target Frameworks:
    • iOS: 13.3
    • Android: 10
  • Android Support Library Version: AndroidX
  • Nuget Packages:
  • Affected Devices: Android

Reproduction Link

SwipeViewFrame.zip

Workaround

4.6.0.967 Seems to work fine for both Android and iOS AFAICT. I couldn't find any other workarounds aside from just removing the gesture recognizer.

@RedChops RedChops added s/unverified New report that has yet to be verified t/bug 🐛 labels Jul 6, 2020
@jsuarezruiz jsuarezruiz self-assigned this Jul 7, 2020
@jsuarezruiz jsuarezruiz added e/2 🕑 2 and removed s/unverified New report that has yet to be verified labels Jul 7, 2020
@samhouts samhouts added the in-progress This issue has an associated pull request that may resolve it! label Jul 7, 2020
@IgorKravchenko10
Copy link

In last known good version sometimes it was hard to scroll CollectionView. Swipe gestures and scroll gestures sometimes clashed with each other.

@darrabam
Copy link

In my project, it works fine in 4.7.0.968. All subsequent builds are not ...

@darrabam
Copy link

darrabam commented Aug 1, 2020

This issue seems to be resolved with 4.7.0.1239. At least in my project where I have a grid with tab gesture, and swipeviews.

@RedChops
Copy link
Author

RedChops commented Aug 2, 2020

@darrabam You seem to be right... Though the tap gesture is a bit dodgy on an emulator. I'll have to try it out on an actual device.

@samhouts @jsuarezruiz I see the PR for this ticket doesn't seem to be merged. Did the fix make it in on a different PR, or did this get resolved incidentally? I don't want to just close this ticket if there's still other work to be done on it.

@TobiasJoergensen
Copy link

I can confirm that version 4.7.0.1239, also resolved the issue for me.

@samhouts samhouts added this to the 5.0.0 milestone Aug 4, 2020
myroot pushed a commit to myroot/Xamarin.Forms that referenced this issue Aug 19, 2020
Co-authored-by: Samantha Houts <samhouts@users.noreply.github.com>
fixes xamarin#11333
sung-su pushed a commit to sung-su/Xamarin.Forms that referenced this issue Aug 20, 2020
Co-authored-by: Samantha Houts <samhouts@users.noreply.github.com>
fixes xamarin#11333
sung-su pushed a commit to sung-su/Xamarin.Forms that referenced this issue Aug 20, 2020
Co-authored-by: Samantha Houts <samhouts@users.noreply.github.com>
fixes xamarin#11333
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a/swipeview e/2 🕑 2 in-progress This issue has an associated pull request that may resolve it! p/Android p/iOS 🍎 t/bug 🐛
Projects
None yet
Development

No branches or pull requests

6 participants