Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: 🐛 DsfrAccordion shouldn't focus on toggle #1010

Conversation

MaitreManuel
Copy link

Bonsoir,

Suite à un bug remonté avec la mise à jour 7.2.0 sur un focus de lien à l'ouverture d'un accordéon et à l'issue 999, je propose ce correctif.

Il permet de, au choix, ne pas focus le premier lien que le composant utilisant le composable onTransitionEnd pourrait posséder.

Dans le cas du DsfrAccordion.vue, il amène un comportement non désiré et provoque une anomalie en terme d'accessibilité en faisant ignorer du contenu par les lecteurs d'écran.

Une très bonne soirée

Copy link

netlify bot commented Jan 6, 2025

Deploy Preview for vue-dsfr-demo ready!

Name Link
🔨 Latest commit e515ec9
🔍 Latest deploy log https://app.netlify.com/sites/vue-dsfr-demo/deploys/677c1d990ae54f0008ef4ca8
😎 Deploy Preview https://deploy-preview-1010--vue-dsfr-demo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@laruiss
Copy link
Collaborator

laruiss commented Jan 11, 2025

Super, merci !

Est-ce que tu veux bien rebaser ta branche sur develop et demander la fusion dans develop ?

@MaitreManuel MaitreManuel changed the base branch from main to develop January 11, 2025 13:00
@MaitreManuel
Copy link
Author

Super, merci !

Est-ce que tu veux bien rebaser ta branche sur develop et demander la fusion dans develop ?

C'est fait !

@laruiss
Copy link
Collaborator

laruiss commented Jan 11, 2025

Merci pour ta réactivité !

@laruiss laruiss merged commit e02e394 into dnum-mi:develop Jan 11, 2025
5 checks passed
@laruiss
Copy link
Collaborator

laruiss commented Jan 11, 2025

Ah, autre chose : le projet étant franco-français, nous avons décidé depuis le début d’écrire les messages de commit en français (en tout cas les fix, feat qui apparaissent dans le release log, et aussi les doc - qui apparaîtront un jour dans le release log aussi). Ainsi que toute doc, ticket, discussion...

@MaitreManuel
Copy link
Author

Pas de soucis, je n'étais pas trop sûr et avec l'habitude j'ai mis en anglais

Copy link

🎉 This PR is included in version 8.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants