mapbox/mapbox-gl-js

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

Open

#9.584 geöffnet am 17. Apr. 2020

Auf GitHub ansehen
 (1 Kommentar) (4 Reaktionen) (1 zugewiesene Person)JavaScript (2.203 Forks)batch import
feature :green_apple:good first issue

Repository-Metriken

Stars
 (10.532 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuche den vorhandenen TouchZoomRotateHandler, um zu verstehen, wie disableRotation/enableRotation implementiert sind. Sieh dir die private Eigenschaft tapDragZoom und ihre disable/enable Methoden an. Erwäge, ähnliche öffentliche Methoden disableTapZoom/enableTapZoom nach demselben Muster hinzuzufügen. Prüfe, ob es laufende Diskussionen in den Issue Kommentaren oder verwandten PRs gibt.
Tech Stack
javascript
Domain
frontend
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitJavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65