streamich/react-use

useEnsuredForwardedRef not working with callback refs

Open

#805 geöffnet am 26. Nov. 2019

Auf GitHub ansehen
 (2 Kommentare) (1 Reaktion) (0 zugewiesene Personen)TypeScript (43.979 Stars) (3.273 Forks)batch import
bughelp wanted

Beschreibung

If the forwarded ref is a callback ref the callback isn't called properly.

useEnsuredForwardedRef should check if the passed ref is a callback-function and if that's the case call it to set the ref.

Contributor Guide

Tech Stack
typescriptreact
Domain
frontend
Issue Type
bug
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.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Understanding React refsFamiliarity with useRef and forwardRef
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85
Research-Richtung
Examine the implementation of useEnsuredForwardedRef in the repository. The hook currently may not handle callback refs. To fix, add a check: if the forwarded ref is a function (callback ref), call it with the value instead of assigning to .current. Write unit tests to verify both object refs and callback refs work correctly.