downshift-js/downshift

Consider using passive event listeners

Open

#1.650 geöffnet am 23. Mai 2025

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (11.761 Stars) (980 Forks)batch import
help wantedneeds investigation

Beschreibung

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?

Contributor Guide

Tech Stack
javascriptreact
Domain
frontendperformance
Issue Type
performance
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.
under 1 hour
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
fresh
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
basic JavaScriptReact hooksevent listeners
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85
Research-Richtung
Focus on the file src/hooks/utils.js around lines 421-425 where event listeners are added. Determine if making these listeners passive (by passing { passive: true } as the third argument to addEventListener) will break any existing functionality, such as preventing default behavior. The issue suggests adding passive listeners to improve scroll performance. Review the related Chrome documentation and test the change with downshift components. If passive listeners are safe, implement and verify there are no regressions.