Merged in fix/BOOK-629-aria-label-social-media (pull request #3259)
fix(BOOK-629): add aria-label to social media on sidepeek * fix(BOOK-629): add aria-label to social media on sidepeek * fix(BOOK-629): add aria-label for social media Approved-by: Erik Tiekstra
This commit is contained in:
@@ -98,13 +98,37 @@ export function ContactInformation({
|
||||
</p>
|
||||
<div className={styles.socialIcons}>
|
||||
{socials.instagram && (
|
||||
<NextLink href={socials.instagram}>
|
||||
<InstagramIcon color="Icon/Interactive/Default" />
|
||||
<NextLink
|
||||
target="_blank"
|
||||
href={socials.instagram}
|
||||
rel="noopener noreferrer"
|
||||
title={intl.formatMessage({
|
||||
id: "common.linkOpenInNewTab",
|
||||
defaultMessage: "Opens in a new tab/window",
|
||||
})}
|
||||
>
|
||||
<InstagramIcon
|
||||
color="Icon/Interactive/Default"
|
||||
// eslint-disable-next-line formatjs/no-literal-string-in-jsx
|
||||
aria-label="Instagram"
|
||||
/>
|
||||
</NextLink>
|
||||
)}
|
||||
{socials.facebook && (
|
||||
<NextLink href={socials.facebook}>
|
||||
<FacebookIcon color="Icon/Interactive/Default" />
|
||||
<NextLink
|
||||
target="_blank"
|
||||
rel="noopener noreferrer"
|
||||
href={socials.facebook}
|
||||
title={intl.formatMessage({
|
||||
id: "common.linkOpenInNewTab",
|
||||
defaultMessage: "Opens in a new tab/window",
|
||||
})}
|
||||
>
|
||||
<FacebookIcon
|
||||
color="Icon/Interactive/Default"
|
||||
// eslint-disable-next-line formatjs/no-literal-string-in-jsx
|
||||
aria-label="Facebook"
|
||||
/>
|
||||
</NextLink>
|
||||
)}
|
||||
</div>
|
||||
|
||||
Reference in New Issue
Block a user