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

Add title for footer links that open in a new tab #153

Closed
slafayIGN opened this issue Jul 28, 2023 · 5 comments
Closed

Add title for footer links that open in a new tab #153

slafayIGN opened this issue Jul 28, 2023 · 5 comments

Comments

@slafayIGN
Copy link
Contributor

This issue has been raised by an accessibility audit on an app using react-dsfr.

The 4 hardcoded links in the footer to "legifrance.gouv.fr", "gouvernement.fr", "service-public.fr" and "data.gouv.fr" open in a new tab (taget="_blank").

They should bear titles like : "legifrance.gouv.fr - ouvre une nouvelle fenêtre"
This is not the case on our app and on the Footer component documentation

Should the Footer component be modified or should it be added by another script ?

@garronej
Copy link
Collaborator

Hi @slafayIGN,
Thanks for rising the issue
I guess the footer component should be modified in this instance. I'll do it this week end.

@slafayIGN
Copy link
Contributor Author

Thanks. The link to the etalab licence has the same issue in this component as well.

@pprev94
Copy link
Contributor

pprev94 commented Aug 9, 2023

I work with Sylvain and I'm working on this issue

@garronej
Copy link
Collaborator

garronej commented Aug 9, 2023

Sorry I forgot to address this.

Thank you for working on this. Your change LGTM, don't forget to open a PR!

sbourdon13 pushed a commit to sbourdon13/react-dsfr that referenced this issue Aug 11, 2023
garronej added a commit that referenced this issue Aug 13, 2023
fix: Add title for footer links that open in a new tab #153
@slafayIGN
Copy link
Contributor Author

Merged and available for versions >= v0.75.0 🚀

🙏

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

No branches or pull requests

3 participants