WordPress/gutenberg

NumberControl: Disable dragging on touch devices

Open

#38.865 aperta il 16 feb 2022

Vedi su GitHub
 (2 commenti) (3 reazioni) (0 assegnatari)JavaScript (3893 fork)batch import
Good First Issue[Package] Components[Status] In Progress[Type] Bug

Metriche repository

Star
 (9607 star)
Metriche merge PR
 (Merge medio 18g 1h) (406 PR mergiate in 30 g)

Descrizione

The drag-to-change-value gesture will not work nicely on touch devices unless we disable browser scrolling with touch-action: none.

There is a separate question of whether we really want this dragging gesture on touch devices, because there is no cursor affordance like on desktop. It’s also pretty easy to accidentally flick and change the value, when you just want to tap/move the caret or something.

I’m inclined to disable the drag gesture on touch devices.

Tested on iPhone (Mobile mode in DevTools has different behavior)

Guida contributor

Direzione di ricerca
Esamina il componente NumberControl nel repository di Gutenberg, probabilmente in packages/components/src/number control/. Il problema suggerisce di aggiungere `touch action: none` per disabilitare lo scorrimento del browser durante il trascinamento e potenzialmente disabilitare condizionalmente il gesto di trascinamento sui dispositivi touch. Controlla i gestori di eventi e il CSS esistenti per il componente. Il problema menziona l'uso della libreria `use gesture`, quindi esamina come è configurato l'evento di trascinamento. Un maintainer potrebbe aver fornito ulteriori input nei commenti.
Tech stack
javascriptreact
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
GitReact
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75