xamarin/Xamarin.Forms

Listview.ItemTapped doesnt fire when viewcell has ContxctAction and GestureRecognizer

Ouverte

#2 026 ouverte le 6 mars 2018

Voir sur GitHubGuide contributeur
 (10 commentaires) (1 réaction) (0 personne assignée)C# (1 926 forks)batch import
a/gestures 🖖a/listviewe/8 :clock8:help wantedp/Androidt/bug :bug:up-for-grabs

Métriques du dépôt

Stars
 (5 644 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Description

ItemTapped event is not fired when a viewcell has contextaction + an element that has a gestrurerecognizer.

On earlier versions of 2.5.0 the issue with this combo was that the contextAction fired just by scrolling the screen (should only fire when longtap), the fix for this issue is here https://github.com/xamarin/Xamarin.Forms/pull/1441.

Now that the fix mentioned is included, Taps doesnt work (with the same combo of viewcell)

Steps to Reproduce

  1. Run Repro (has instructions and is extremely basic)

Expected Behavior

Tapevent should fire

Actual Behavior

Tapevent doesnt fire

Basic Information

  • Version with issue: 2.5.1-pre1
  • Last known good version: 2.4.X
  • IDE: Visual Studio 2017
  • Affected Devices: Android (only tested on android)

Reproduction Link

listtap.zip

Guide contributeur

Direction de recherche
Reproduisez le problème à l'aide de l'échantillon fourni, tracez le flux d'événements pour ItemTapped et les reconnaisseurs de gestes, et identifiez où l'événement est supprimé.
Stack technique
csharpandroid
Domaine
mobile
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#Xamarin.Forms
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65