mapbox/mapbox-gl-js

v1.10.0-beta.1 - No way to disable new "tap then drag" to zoom gesture

Open

#9.584 aberto em 17 de abr. de 2020

Ver no GitHub
 (1 comment) (4 reactions) (1 assignee)JavaScript (2.203 forks)batch import
feature :green_apple:good first issue

Métricas do repositório

Stars
 (10.532 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

The TouchZoomRotateHandler support various gestures ("zoom rotate", "pinch zoom/touch zoom", "single finger zoom/tap then drag"), however it only allows you to disable the "zoom rotate" gesture.

The individual gestures can be disabled by diving in to the private members (such as by calling "map.touchZoomRotate._tapDragZoom.disable();").

It would be nicer to have a match for disableRotation()/enableRotation() to turn on/off each supported gesture (i.e. add a disableTapZoom()/enableTapZoom()).

Guia do colaborador

Direção de pesquisa
Examine o TouchZoomRotateHandler existente para entender como disableRotation/enableRotation são implementados. Observe a propriedade privada tapDragZoom e seus métodos disable/enable. Considere adicionar métodos públicos semelhantes disableTapZoom/enableTapZoom seguindo o mesmo padrão. Verifique se há discussões em andamento nos comentários da issue ou em PRs relacionados.
Pilha de tecnologia
javascript
Domain
frontend
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
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
GitJavaScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65