react-dsfr
react-dsfr copied to clipboard
codegouvfr
Component Range : la props "step" fait crasher le navigateur sur le storybook et dans l'app
Le component <Range /> crash lors de l'utilisation de la props step sur navigateur chrome et Edge (sans step, le component fonctionne normalement), vous pouvez facilement reproduire ce bug directement via le storybook si vous cliquer sur Set Number : https://components.react-dsfr.codegouv.studio/?path=/docs/components-range--default
Pour reproduire ->
Ne crash pas :
<Range
label="Dans un rayon de"
max={100}
min={0}
suffix=" km"
/>
Crash :
<Range
label="Dans un rayon de"
max={100}
min={0}
step={25}
suffix=" km"
/>
il semblerait que cela ait été corrigé dans ce commit sur @gouvfr/dsfr en 1.11.2; Or react-dsfr utilise encore la 1.11.1
il faudrait qu'une bonne âme tente d'upgrade ici la dépendance à @gouvfr/dsfr
Oui j'ai également bumper la version en local mais le soucis ne vient pas de là , dès qu'on fournit un "step" ça crash :(
Je vois que @ddecrulle a commenté la story associé au step pour ne pas crasher le storybook, peut-être as-tu trouvé quelque chose de ton côté ? :)
il faudrait qu'une bonne âme tente d'upgrade ici la dépendance à @gouvfr/dsfr
Attend ça c'est trivial je te le fait tout de suite!
Done! @revolunet Tu sais c'est facil, il suffit juste de bump la version de @gouvfr/dsfr dans le package.json.
Si il y a des breaking change le transpilateur CSS va donner du rouge.
Si il y a des régression visuelle je peux les voir au niveau du storybook avec chromatic en théorie. J'ai juste pas réussi a configurer Chromatique pour un projet open source.
Ils nous on donner un accès gratuit mais je ne sais pas comment faire une sorte de ne pas leaker le token...
Anyway
Thanks @garronej 🤗 J'ai testé de bump mais j'avais un souci dans le storybook qui freeze sur le composant Range... Je reteste dans la journée
Hello désolé j'avais pas vu ce thread, c'est un bug du dsfr il faut attendre que cette issue https://github.com/GouvernementFR/dsfr/issues/865 soit corrigée
