Skip to content

Commit

Permalink
Merge pull request #681 from Financial-Times/CI-1362-DAC_Incorrect_Ar…
Browse files Browse the repository at this point in the history
…ia_Label_01

[CI-1362] show title in arial-label attribute for every share button
  • Loading branch information
AlbertoCuberoNavas authored Nov 16, 2022
2 parents 71f6774 + f69dd28 commit 4adb3de
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions components/x-live-blog-post/src/ShareButtons.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,8 @@ export default ({ postId, articleUrl, title }) => {
href={twitterUrl}
data-trackable="twitter"
>
<span className="o-share__text" aria-label={`Share ${title} on Twitter`}>
Share on Twitter (opens new window)
<span className="o-share__text">
Share ${title} on Twitter (opens in a new window)
</span>
</a>
</li>
Expand All @@ -43,8 +43,8 @@ export default ({ postId, articleUrl, title }) => {
href={facebookUrl}
data-trackable="facebook"
>
<span className="o-share__text" aria-label={`Share ${title} on Facebook`}>
Share on Facebook (opens new window)
<span className="o-share__text">
Share ${title} on Facebook (opens in a new window)
</span>
</a>
</li>
Expand All @@ -55,8 +55,8 @@ export default ({ postId, articleUrl, title }) => {
href={linkedInUrl}
data-trackable="linkedin"
>
<span className="o-share__text" aria-label={`Share ${title} on LinkedIn`}>
Share on LinkedIn (opens new window)
<span className="o-share__text">
Share ${title} on LinkedIn (opens in a new window)
</span>
</a>
</li>
Expand Down

0 comments on commit 4adb3de

Please sign in to comment.