mapbox/mapbox-gl-js

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

Open

#9584 aperta il 17 apr 2020

Vedi su GitHub
 (1 commento) (4 reazioni) (1 assegnatario)JavaScript (2203 fork)batch import
feature :green_apple:good first issue

Metriche repository

Star
 (10.532 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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()).

Guida contributor

Direzione di ricerca
Esamina l'esistente TouchZoomRotateHandler per capire come sono implementati disableRotation/enableRotation. Osserva la proprietà privata tapDragZoom e i suoi metodi disable/enable. Considera l'aggiunta di metodi pubblici simili disableTapZoom/enableTapZoom seguendo lo stesso schema. Controlla se ci sono discussioni in corso nei commenti dell'issue o nelle PR correlate.
Tech stack
javascript
Dominio
frontend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65