xamarin/Xamarin.Forms

Listview.ItemTapped doesnt fire when viewcell has ContxctAction and GestureRecognizer

Aberta

#2.026 aberto em 6 de mar. de 2018

Ver no GitHubGuia do colaborador
 (10 comentários) (1 reação) (0 responsável)C# (1.926 forks)batch import
a/gestures 🖖a/listviewe/8 :clock8:help wantedp/Androidt/bug :bug:up-for-grabs

Métricas do repositório

Stars
 (5.644 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

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

Guia do colaborador

Direção de pesquisa
Reproduza o problema usando a amostra fornecida, rastreie o fluxo de eventos para ItemTapped e reconhecedores de gestos, e identifique onde o evento está sendo suprimido.
Pilha de tecnologia
csharpandroid
Domain
mobile
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C#Xamarin.Forms
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65