Skip to content

Commit 87c51a5

Browse files
committed
Merge pull request #2975 from PrakashDurlabhji/issue_2789
Issue 2789 fix
1 parent bdab770 commit 87c51a5

File tree

2 files changed

+8
-4
lines changed

2 files changed

+8
-4
lines changed

__tests__/shared/components/__snapshots__/TopcoderFooter.jsx.snap

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,7 @@ exports[`Matches shallow shapshot 1`] = `
5858
>
5959
<hr />
6060
<a
61+
aria-label="Facebook"
6162
href="https://www.facebook.com/topcoder"
6263
target="fbwindow"
6364
>
@@ -78,6 +79,7 @@ exports[`Matches shallow shapshot 1`] = `
7879
</svg>
7980
</a>
8081
<a
82+
aria-label="Twitter"
8183
href="http://www.twitter.com/topcoder"
8284
target="twwindow"
8385
>
@@ -98,6 +100,7 @@ exports[`Matches shallow shapshot 1`] = `
98100
</svg>
99101
</a>
100102
<a
103+
aria-label="Linkedin"
101104
href="https://www.linkedin.com/company/topcoder"
102105
target="liwindow"
103106
>
@@ -118,6 +121,7 @@ exports[`Matches shallow shapshot 1`] = `
118121
</svg>
119122
</a>
120123
<a
124+
aria-label="Instagram"
121125
href="https://www.instagram.com/topcoder"
122126
target="inwindow"
123127
>

src/shared/components/TopcoderFooter/index.jsx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -55,16 +55,16 @@ TERMS
5555
</nav>
5656
<div styleName="social-links">
5757
<hr />
58-
<a href="https://www.facebook.com/topcoder" target="fbwindow">
58+
<a href="https://www.facebook.com/topcoder" target="fbwindow" aria-label="Facebook">
5959
<FacebookIcon />
6060
</a>
61-
<a href="http://www.twitter.com/topcoder" target="twwindow">
61+
<a href="http://www.twitter.com/topcoder" target="twwindow" aria-label="Twitter">
6262
<TwitterIcon />
6363
</a>
64-
<a href="https://www.linkedin.com/company/topcoder" target="liwindow">
64+
<a href="https://www.linkedin.com/company/topcoder" target="liwindow" aria-label="Linkedin">
6565
<LinkedInIcon />
6666
</a>
67-
<a href="https://www.instagram.com/topcoder" target="inwindow">
67+
<a href="https://www.instagram.com/topcoder" target="inwindow" aria-label="Instagram">
6868
<InstagramIcon />
6969
</a>
7070
</div>

0 commit comments

Comments
 (0)