Skip to content

Commit 5d19255

Browse files
author
Trystan EVENO
committed
fix (DsfrShare): 🐛 ajout mention 'nouvelle fenetre' pour eviter la redondance du lien 'Envoyer un mail'
1 parent b43e0c6 commit 5d19255

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

src/components/DsfrShare/DsfrShare.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ describe('DsfrShare', () => {
88
const copyLabel = 'Label de la copie dans le presse-papier'
99
const mail = {
1010
to: 'mailto:?subject=[À MODIFIER - objet du mail]&body=[À MODIFIER - titre ou texte descriptif de la page] [À MODIFIER - url de la page]',
11-
label: 'Partager par mail personnalisé',
11+
label: 'Partager par mail personnalisé - nouvelle fenêtre',
1212
}
1313
const networks = [
1414
{
@@ -47,7 +47,7 @@ describe('DsfrShare', () => {
4747
}
4848

4949
const mailToLinkElement = getByText(mail.label)
50-
expect(mailToLinkElement).toHaveAttribute('title', mail.label)
50+
expect(mailToLinkElement).toHaveAttribute('title', `${mail.label} - nouvelle fenêtre`)
5151
expect(mailToLinkElement).toHaveAttribute('href', mail.to)
5252

5353
const copyButtonElement = getByText(copyLabel)

src/components/DsfrShare/DsfrShare.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ const openWindow = ({ url, label }: { url: string, label: string }) => {
4949
<a
5050
class="fr-btn fr-btn--mail"
5151
:href="mail.to"
52-
:title="mail.label"
52+
:title="`${mail.label} - nouvelle fenêtre`"
5353
target="_blank"
5454
rel="noopener noreferrer"
5555
>

0 commit comments

Comments
 (0)