8000 Accessibilité des liens externes by LucienMLD · Pull Request #277 · numerique-gouv/sites-faciles · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Accessibilité des liens externes #277

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

Merged
merged 3 commits into from
Mar 3, 2025
Merged

Accessibilité des liens externes #277

merged 3 commits into from
Mar 3, 2025

Conversation

LucienMLD
Copy link
Member

🎯 Objectif

Quand un lien redirige vers une url externe il n'y a qu'une information présente dans un ::after qui n'est donc pas visible quand il n'y a pas de feuille de style d'activé.

Critère 10.2 Dans chaque page web, le contenu visible (ouvre dans une nouvelle fenêtre)porteur d’information reste-t-il présent lorsque les feuilles de styles (ouvre dans une nouvelle fenêtre)sont désactivées ?

🔍 Implémentation

  • ajouter une information "Ouvre dans une nouvelle fenêtre" avec une class "sr-only"

⚠️ Informations supplémentaires

(optionnel) Documentation, commandes à lancer, variables d’environnement, etc.

🏕 Amélioration continue

  • (optionnel) Une liste d’autres modifications pas en lien direct avec la PR

🖼️ Images

Capture d’écran 2025-02-28 à 15 16 55
Avant :
Capture d’écran 2025-02-28 à 15 16 49
Après :
Capture d’écran 2025-02-28 à 15 18 28

@Ash-Crow Ash-Crow changed the title accessibilité des liens externe Accessibilité des liens externes Mar 3, 2025
@Ash-Crow
Copy link
Collaborator
Ash-Crow commented Mar 3, 2025

Merci ! J'ai ajouté l'internationalisation :)

@Ash-Crow Ash-Crow merged commit 7407df9 into main Mar 3, 2025
2 of 10 checks passed
@Ash-Crow Ash-Crow deleted the lucien/liens-externes branch March 3, 2025 10:26
@LucienMLD
Copy link
Member Author

Merci ! J'ai ajouté l'internationalisation :)

Bien vu j'avais oublié

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

Successfully merging this pull request may close these issues.

2 participants
0