search

dnum-mi / vue-dsfr

Portage en Vue du DSFR
https://vue-ds.fr
Other
58 stars 27 forks source link

Soucis avec les accordéons #271

Closed LVerneyPEReN closed 2 years ago

LVerneyPEReN commented 2 years ago

Bonjour,

En testant vue-dsfr, j'ai rencontré quelques soucis avec les accordéons :

  1. D'une part, la méthode d'installation dans le README installe la version beta37 qui ne correspond pas à la doc en ligne (notamment sur la gestion des expanded) : https://github.com/dnum-mi/vue-dsfr/blob/develop/README.md#comment-lutiliser
  2. La doc pour l'élément DsfrAccordion n'est pas fonctionnelle : https://vue-dsfr.netlify.app/?path=/docs/composants-accord%C3%A9on-dsfraccordion--accordeon
  3. Sur https://vue-dsfr.netlify.app/?path=/docs/composants-accord%C3%A9on-dsfraccordionsgroup--accordeon-groupe, les éléments expanded{1,2,3} n'ont aucun effet.
  4. Sur https://vue-dsfr.netlify.app/?path=/docs/composants-accord%C3%A9on-dsfraccordionsgroup--accordeon-groupe, le bouton "show-code" affiche un code présentant une typo : :expanded-id qui devrait être :expandedId.

Merci !

laruiss commented 2 years ago
  1. Tout à fait, une nouvelle version de create-vue-dsfr devrait sortir très prochainement (aujourd’hui ou demain)
  2. En effet, merci de nous le signaler. Nous sommes en train de la corriger
  3. Nous allons le corriger aussi.
  4. Ce n’est pas une typo, les props de Vue peuvent être notées en kebab-case ou camelCase, et il est recommandé d’utiliser le kebab-case
laruiss commented 2 years ago

Voilà, tout est corrigé dans la beta.43 et sur le storybook

LVerneyPEReN commented 2 years ago

Merci pour la correction rapide !

Au passage, deux remarques en coup de vent sur la doc et le bouton "show code" :

laruiss commented 2 years ago

Malheureusement, je pense que ce sont des bugs de Storybook et qu’on ne pourra pas y faire grand chose...

Pour l’exemple de Accordion, il vaut mieux tenir compte du code présent dans la story DsfrAccordionGroup.

Merci de les signaler, et merci pour vos retours !