downshift-js/downshift

Consider using passive event listeners

Ouverte

#1 650 ouverte le 23 mai 2025

Voir sur GitHubGuide contributeur
 (2 commentaires) (0 réaction) (0 personne assignée)JavaScript (980 forks)batch import
help wantedneeds investigation

Métriques du dépôt

Stars
 (11 761 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

When rendering a Downshift component in Chrome, I see this message logged in the console:

[Violation] Added non-passive event listener to a scroll-blocking 'touchstart' event. Consider marking event handler as 'passive' to make the page more responsive. See https://www.chromestatus.com/feature/5745543795965952

Chrome developer tools indicates that these listeners are not passive: https://github.com/downshift-js/downshift/blob/master/src/hooks/utils.js#L421-L425

Would it be possible to make these listeners passive without affecting functionality?

Guide contributeur

Direction de recherche
Regardez les enregistrements d'écouteurs d'événements dans src/hooks/utils.js lignes 421-425 et ajoutez l'option { passive: true } là où cela ne casse pas la fonctionnalité.
Stack technique
javascriptreact
Domaine
frontend
Type d'issue
Performance
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
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
JavaScriptReact
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90