search

codegouvfr / react-dsfr

🇫🇷 The Frech Government Design system React toolkit
https://react-dsfr.codegouv.studio
MIT License
403 stars 49 forks source link

Component Range : la props "step" fait crasher le navigateur sur le storybook et dans l'app #254

Closed ThomasDos closed 2 months ago

ThomasDos commented 4 months ago

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"
        />
revolunet commented 4 months ago

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

revolunet commented 4 months ago

bon apparemment ca ne suffit pas il y un truc buggy dans ce composant :/

ThomasDos commented 4 months ago

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 :(

ThomasDos commented 4 months ago

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é ? :)

garronej commented 4 months ago

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!

garronej commented 4 months ago

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

revolunet commented 4 months ago

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

ddecrulle commented 3 months ago

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

garronej commented 3 months ago

Merci pour le followup

ddecrulle commented 2 months ago

La version 1.12.0 du dsfr corrige ce souci, disponible dans react-dsfr@1.9.20.

revolunet commented 2 months ago

thanks @ddecrulle 👍