feat: ✨ Suppression du clic sur le label et sur le message d'erreur du DsfrFileUpload #997
Conversation
❌ Deploy Preview for vue-dsfr failed.
|
✅ Deploy Preview for demo-vue-dsfr ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for docs-vue-dsfr ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
| @@ -51,6 +51,7 @@ const acceptTypes = computed(() => { | |||
| <label | |||
| class="fr-label" | |||
| :for="id" | |||
| @click.prevent | |||
There was a problem hiding this comment.
Ceci ne me semble pas accessible. Pourquoi tu penses que c’est souhaitable ?
There was a problem hiding this comment.
C'est une remontée de bug chez nous. On voudrait que la fenêtre permettant de sélectionner le/les fichiers soit ouverte uniquement au click sur le input type file (le bouton parcourir). Actuellement la fenêtre s'ouvre aussi au click sur le label de l'input et au click sur le message d'erreur de l'input.
Quel problème cela pose pour l'accessibilité ? J'ai essayé la navigation au clavier et cela fonctionne toujours avec l'ajout des @click.prevent.
There was a problem hiding this comment.
Peut être qu'on pourrait conditionner le click.prevent en fonction d'une nouvelle props ?
There was a problem hiding this comment.
Je ne vois pas pourquoi ce serait considéré comme un bug. C’est une vraie feature. C’est bien prévu comme cela dans le DSFR, notamment parce que cliquer sur le bouton "parcourir" demande de la précision, et il est, pour certaines personnes, difficile de cliquer exactement sur le bouton parcourir. Je ne le rajouterai donc pas.
There was a problem hiding this comment.
OK tu as raison, on laisse comme ça.
| @@ -79,6 +80,7 @@ const acceptTypes = computed(() => { | |||
| v-if="error || validMessage" | |||
| class="fr-messages-group" | |||
| role="alert" | |||
| @click.prevent | |||
There was a problem hiding this comment.
Voir commentaire précédent.
There was a problem hiding this comment.
Ici, en revanche, ce n’est pas normal que cela déclenche l’événement. Et ce n’est pas le cas dans l’exemple du Storybook : https://sb.vue-ds.fr/?path=/story/composants-dsfrfileupload--televersement-avec-erreur
Je serait tenté de penser à une erreur d’imbrication de composant dans votre application.
…'erreur du DsfrFileUpload