-
Notifications
You must be signed in to change notification settings - Fork 46
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Régression style fr-input #276
Comments
Hello, l'objectif de la PR et surtout de pouvoir surcharger Deux solutions, soit tu retires le className des props passées par Formik comme j'ai fais, soit on considère qu'il n'est jamais nécessaire de changer le classname (de retirer |
Pour les props passées par Formik, on ne passe aucune class, c'est Formik qui passe className, même si c'est undefined. Et je suis pas sûr de vouloir modifier Formik là... 🙃 |
Il ne s'agit pas de modifier Formik ... simplement enlever le className des props générées par Formik avant de spread ça au composant. J'ai tout de même revert la PR. |
💥 Il semblerait que la PR #274 ait déclenché une régression de notre côté.
On utilise Formik qui passe automatiquement des propriétés au composant Input dont className ce qui fait que la classe fr-input et les classes de validation sautent.
Problème visible ici : il faut faire un test d'adresse pour afficher le formulaire qui pose problème.
J'ai pas le temps de regarder pour un correctif ce soir donc ça attendra la semaine prochaine au minimum. Et aussi je ne connais pas le cas d'usage pour lequel a été créé la PR... 🤷
Un avis ? Est-ce que l'objectif de la PR était de surcharger complètement className par exemple ou juste d'ajouter des classes ?
The text was updated successfully, but these errors were encountered: