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.

Sign up for GitHub

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

Jump to bottom

Ajout de liens de recours dans la déclaration #812

Merged
merged 3 commits into from
Oct 18, 2024
Merged

Ajout de liens de recours dans la déclaration #812

merged 3 commits into from
Oct 18, 2024

Conversation

bellangerq
Copy link
Collaborator

closes #811

@hissalht hissalht temporarily deployed to ara-811-ajouter-des-lie-gejycl October 9, 2024 12:39 Inactive
benoitdequick
benoitdequick reviewed Oct 9, 2024
View reviewed changes
Copy link
Contributor

@benoitdequick benoitdequick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

possible d'ajouter un attribut title="Écrire un message au Défenseur des droits - nouvelle fenêtre" et title="Contacter le délégué ... - nouvelle fenêtre" ?

@bellangerq
Copy link
Collaborator Author

bellangerq commented Oct 9, 2024 •
edited by benoitdequick
Loading

possible d'ajouter un attribut title="Écrire un message au Défenseur des droits - nouvelle fenêtre" et title="Contacter le délégué ... - nouvelle fenêtre" ?

Je sais jamais ce qui est le mieux : le title qui reprend tout l'intitulé + "nouvelle fenêtre" ou la span cachée "nouvelle fenêtre" ?

Les deux sont ok. Il faut juste être cohérent et adopter la même technique sur l'ensemble du site. Et j'ai l'impression que le span l'emporte ;)

@bellangerq bellangerq force-pushed the 811-ajouter-des-liens-dans-la-page-de-rédaction-de-la-déclaration-da11é branch from fa7922a to 86483b5 Compare October 18, 2024 08:36
@hissalht hissalht temporarily deployed to ara-811-ajouter-des-lie-gejycl October 18, 2024 08:36 Inactive
@bellangerq bellangerq merged commit 0bf603a into main Oct 18, 2024
1 check passed
@bellangerq bellangerq deleted the 811-ajouter-des-liens-dans-la-page-de-rédaction-de-la-déclaration-da11é branch October 18, 2024 08:36
@yaaax
Copy link
Collaborator

yaaax commented Nov 15, 2024

Je sais jamais ce qui est le mieux : le title qui reprend tout l'intitulé + "nouvelle fenêtre" ou la span cachée "nouvelle fenêtre" ?

Les deux sont ok. Il faut juste être cohérent et adopter la même technique sur l'ensemble du site. Et j'ai l'impression que le span l'emporte ;)

Salut @bellangerq,

Le DSFR utilise plutôt la version title. Mais c’est vrai qu’en retirant les CSS on perd l’info quand on ne peut utiliser une souris…
J’observe que pour l’instant sur Ara on a plutôt utilisé la technique <span>. OK pour me caler sur ça !

Pour info, chez gov.uk on préconise tout simplement “Avoid opening links in a new tab or window“
Opening links in a new tab – Links – GOV.UK Design System.
Mais la technique du <span> est effectivement autorisée (pas celle du title)

@yaaax
Copy link
Collaborator

yaaax commented Nov 15, 2024

Je viens de créer #878 ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@benoitdequick benoitdequick benoitdequick approved these changes

@hissalht hissalht hissalht approved these changes

@yaaax yaaax Awaiting requested review from yaaax

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Ajouter des liens dans la page de rédaction de la déclaration d'a11é
4 participants
@bellangerq @yaaax @hissalht @benoitdequick