dnum-mi / vue-dsfr

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

[DSFR Tabs] Uncaught TypeError: event is undefined #827

Closed alemangui closed 3 weeks ago

alemangui commented 3 months ago

Bonjour,

En utilisant le composant DsfrTabs, si on utilise la touche Tab du clavier pour rendre le focus dans l'onglet d'après, une exception se lève car l'event levé est undefined.

C'est assez simple de le reproduire en allant sur https://vue-dsfr.netlify.app/?path=/docs/composants-dsfrtabs--docs, si on navigue sur les exemples en utilisant la tabulation on a cette erreur :

image

laruiss commented 3 months ago

Constatez-vous l’erreur ici https://vue-ds.fr/composants/DsfrTabs ? La touche tab n’amène pas à l’onglet suivant, cela amène au contenu de l’onglet actif. Pour naviguer d’onglet en onglet, il faut utilser les flèches. Exactement comme le DSFR (https://www.systeme-de-design.gouv.fr/elements-d-interface/composants/onglet)

alemangui commented 3 months ago

Merci pour votre réponse @laruiss, en effet sur vue-ds.fr le comportement n'est pas le même.

Je ne suis pas sur de comprendre la différence entre les deux sites de documentation, s'agit-il des mêmes versions de la lib ? faut-il plutôt que je privilégie vue-ds.fr ?

laruiss commented 3 months ago

C'est la même bibliothèque et la petite version (la dernière publiée) qui est utilisée sur les deux sites. Comme noté dans le storybook, il est préférable d'utiliser la nouvelle documentation plutôt que le Storybook. Avec storybook, le code de demo n'est pas aussi isolé que dans la nouvelle doc.

Il manque encore la documentation de quelques composants sur la nouvelle doc.

alemangui commented 3 months ago

@laruiss au temps pour moi, le bug est bien présent sur la nouvelle doc aussi - c'est juste caché dans la console. Le t fait référence à l'event :

image

laruiss commented 3 months ago

Très bien, merci de l'avoir signalé. Je regarde cela dès que possible !

laruiss commented 3 weeks ago

Corrigé dans v5.18.0