WordPress/gutenberg

NumberControl: Disable dragging on touch devices

Open

#38 865 ouverte le 16 févr. 2022

Voir sur GitHub
 (2 commentaires) (3 réactions) (0 assignés)JavaScript (3 893 forks)batch import
Good First Issue[Package] Components[Status] In Progress[Type] Bug

Métriques du dépôt

Stars
 (9 607 stars)
Métriques de merge PR
 (Merge moyen 12j 18h) (509 PRs mergées en 30 j)

Description

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)

Guide contributeur

Direction de recherche
Examinez le composant NumberControl dans le dépôt Gutenberg, probablement dans packages/components/src/number control/. Le problème suggère d'ajouter `touch action: none` pour désactiver le défilement du navigateur pendant le glisser déposer, et potentiellement désactiver conditionnellement le geste de glisser sur les appareils tactiles. Vérifiez les gestionnaires d'événements et le CSS existants pour le composant. Le problème mentionne l'utilisation de la bibliothèque `use gesture`, donc examinez comment l'événement de glisser est configuré. Un mainteneur peut avoir fourni des informations supplémentaires dans les commentaires.
Stack technique
javascriptreact
Domaine
frontend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitReact
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75