GouvernementFR / dsfr

🇫🇷 Official french government's design system (Système de Design de l'État)
https://www.systeme-de-design.gouv.fr/
Other
243 stars 54 forks source link

Deux collapse dans le menu ne fonctionne pas quand il passe en mobile #789

Closed mfo closed 3 months ago

mfo commented 1 year ago

Décrire le bug

Lorsqu'il y a plus d'un .fr-collapse dans le burger menu, seul le dernier se deplie sur mobile.

Les étapes pour reproduire le bug

Exemple :

  1. Faire un header avec deux (par exemple) deux composant de traducteur de langue
  2. Passer sur mobile
  3. Cliquer sur le premier, RAS
  4. Cliquer sur le 2eme, ça se déplie

Comportement attendu

Que les deux .fr-collapse fonctionne

Capture d’écran

J'ai pas ça

Configuration et système utilisé

Informations complémentaires

C'est par la que ça se passe : https://github.com/GouvernementFR/dsfr/blob/main/src/component/header/script/header/header-links.js#L19 ; j'imagine que la regexp a du mal

mfo commented 1 year ago

Si ça peut aider, on a contourné avec ce patch qui fait de la modif de dom plutôt que de la regexp : https://github.com/demarches-simplifiees/demarches-simplifiees.fr/pull/9476

keryanS commented 3 months ago

Correction apportée par la version 1.11.0