jaredpalmer/formik

Use React.forwardRef in <Formik>

Open

#2 208 ouverte le 14 janv. 2020

Voir sur GitHub
 (14 commentaires) (4 réactions) (0 assignés)TypeScript (34 255 stars) (2 801 forks)batch import
Component: FormikFocus: TypeScripthelp wanted

Description

As of 2.1.2, <Formik> now allows for an innerRef prop. However, we should instead use React.forwardRef + React.useImperativeHandle. This is more intuitive and closer to best practices. The problem though, is that <Formik> takes TypeScript generics for Values and I'm not sure how to express this in with forwardRef. AFAICT, you can't because forwardRef returns a constant and constants can't have generics.

Help?!?!

Guide contributeur

Stack technique
reacttypescript
Domaine
frontend
Type d'issue
refactor
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
half day
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
needs maintainer response
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
mostly clear
Prérequis
ReactTypeScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40
Direction de recherche
This issue requests converting the <Formik> component to use React.forwardRef. The main challenge is handling TypeScript generics for the Values type parameter. The relevant file is likely src/Formik.tsx. The maintainer should clarify how to type forwardRef with generic props, possibly using React.ForwardRefRenderFunction<HTMLDivElement, FormikProps<Values>>.