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

[Bug] TapGestureRecognizer Not working on Path on iOS #13020

Closed
ColeXm opened this issue Dec 2, 2020 · 3 comments · Fixed by #11419
Closed

[Bug] TapGestureRecognizer Not working on Path on iOS #13020

ColeXm opened this issue Dec 2, 2020 · 3 comments · Fixed by #11419
Assignees
Labels
a/gestures 🖖 a/shapes in-progress This issue has an associated pull request that may resolve it! t/bug 🐛
Projects

Comments

@ColeXm
Copy link

ColeXm commented Dec 2, 2020

Description

TapGestureRecognizer(setting on Path) does not work on iOS but it works on Android .

Steps to Reproduce

  1. Create a simple Path and add TapGestureRecognizer
  <Path Data="M 10,100 L 100,100 100,50Z"
      Stroke="Black"
      StrokeThickness="5"
      Aspect="Uniform"
      HorizontalOptions="Start" >
        <Path.GestureRecognizers>
            <TapGestureRecognizer Tapped="TapGestureRecognizer_Tapped"/>
        </Path.GestureRecognizers>
    </Path>

  1. Click on the path .

Expected Behavior

TapGestureRecognizer_Tapped event triggers

Actual Behavior

TapGestureRecognizer_Tapped event not trigger

Basic Information

Xamarin.Forms : 4.8.0.1687
Xamarin.iOS : 14.4.1.3

Test on iPhone 8 simulator

@ColeXm ColeXm added s/unverified New report that has yet to be verified t/bug 🐛 labels Dec 2, 2020
@samhouts samhouts added this to New in Triage Dec 2, 2020
@jsuarezruiz jsuarezruiz self-assigned this Dec 2, 2020
@jsuarezruiz jsuarezruiz moved this from New to Ready For Work in Triage Dec 2, 2020
@jsuarezruiz jsuarezruiz removed the s/unverified New report that has yet to be verified label Dec 2, 2020
@jsuarezruiz
Copy link
Contributor

Is the same as #10623 and will be fixed by #11419

@amura11
Copy link

amura11 commented Dec 2, 2020

@jsuarezruiz sorry if this is answered somewhere else, I couldn't find it myself, is there any timeline on when 5.0.0 will be released? The workaround mentioned in #10623 didn't work for me and I'm in a bit of a time crunch. If 5.0.0 won't be available in the near future I can look into an alternative approach (maybe just use an image button for now)

@jsuarezruiz jsuarezruiz added the in-progress This issue has an associated pull request that may resolve it! label Dec 4, 2020
@jsuarezruiz
Copy link
Contributor

@amura11 In the next weeks. I can let you know when having a specific date.

Triage automation moved this from Ready For Work to Closed Dec 4, 2020
rmarinho added a commit that referenced this issue Dec 4, 2020
…ixes #13020

* Added repro sample

* Fixed the issue

* Update Issue11050.xaml.cs

* Updated issue 10626. Replaced Color by Brush.

* Fixed build error

* Fixed build error

* Added Issue12685 test

* Changes in text format

* Fix rebase error

Co-authored-by: Samantha Houts <samhouts@users.noreply.github.com>
Co-authored-by: E.Z. Hart <hartez@gmail.com>
Co-authored-by: Rui Marinho <me@ruimarinho.net>
pictos pushed a commit to pictos/Xamarin.Forms that referenced this issue Dec 30, 2020
…ixes xamarin#12685 fixes xamarin#13020

* Added repro sample

* Fixed the issue

* Update Issue11050.xaml.cs

* Updated issue 10626. Replaced Color by Brush.

* Fixed build error

* Fixed build error

* Added Issue12685 test

* Changes in text format

* Fix rebase error

Co-authored-by: Samantha Houts <samhouts@users.noreply.github.com>
Co-authored-by: E.Z. Hart <hartez@gmail.com>
Co-authored-by: Rui Marinho <me@ruimarinho.net>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a/gestures 🖖 a/shapes in-progress This issue has an associated pull request that may resolve it! t/bug 🐛
Projects
Triage
  
Closed
Development

Successfully merging a pull request may close this issue.

3 participants