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 : BUG avec l'option double #259

Closed pprev94 closed 2 months ago

pprev94 commented 2 months ago

Le composant Range en mode double ne se comporte pas comme il devrait. En effet lorsque les deux valeurs min et max sont identiques et que l'on tire par exemple le handle de gauche vers la droite, les deux handles changent de positions mais seule le onChange de la valeur min est déclenché, c'est très gênant

garronej commented 2 months ago

@pprev94 merci pour le rapport.

@lsagetlethias tu pense que c'est un bug de @gouvfr/dsfr ou que c'est a nous de corriger?

lsagetlethias commented 2 months ago

Le problème vient du côté du DSFR car de notre côté on forward bien les deux nativeInputProps.onChange. Je suppose qu'il ne déclenche pas le trigger du second onChange lorsque que le premier handler "pousse" le second.

image

Comme le montre ce screen, même en natif le trigger ne se fait pas, uniquement la propriété value est mise à jour pour l'input "poussé".

Peut être il y'a des events customs de trigger mais rien ne l'indique ni dans la doc (https://www.systeme-de-design.gouv.fr/utilisation-et-organisation/developpeurs/api-javascript) ni dans le code.

garronej commented 2 months ago

Hello @lsagetlethias,

Merci d'avoir regarder! Je vais faire remonter.

ddecrulle commented 2 months ago

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