Skip to content
This repository was archived by the owner on Feb 5, 2022. It is now read-only.

Commit 4ee6a28

Browse files
authored
Merge pull request #476 from openforge/fix/accesibility-link-name
fix(footer): add accessibility links aria label
2 parents 7a5c591 + 3aa4451 commit 4ee6a28

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

src/components/app-footer/app-footer.tsx

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,9 @@ export class AppFooter {
2323
<br />
2424
{translate('footer.address.phoneNumber')}
2525
<br />
26-
<a href="mailto:hello@openforge.io" target="_blank">{translate('footer.address.email')}</a>
26+
<a href="mailto:hello@openforge.io" target="_blank">
27+
{translate('footer.address.email')}
28+
</a>
2729
<br />
2830
<br class="d-none d-md-block" />
2931
<stencil-route-link url="/service-level-agreement">{translate('footer.social.sla')}</stencil-route-link>
@@ -56,16 +58,16 @@ export class AppFooter {
5658
<div class="col-lg-6 col-md-6 col-sm-12 order-first order-md-2 social">
5759
<p class="follow">
5860
Follow us:
59-
<a href="https://www.instagram.com/openforgemobile/" id="instagram" target="_blank" rel="noopener">
61+
<a href="https://www.instagram.com/openforgemobile/" id="instagram" target="_blank" rel="noopener" aria-label="Openforge Instagram account link">
6062
<i class="fab fa-instagram" aria-hidden="true" />
6163
</a>
62-
<a href="https://twitter.com/openforgemobile" id="twitter" target="_blank" rel="noopener">
64+
<a href="https://twitter.com/openforgemobile" id="twitter" target="_blank" rel="noopener" aria-label="Openforge Twitter account link">
6365
<i class="fab fa-twitter-square" aria-hidden="true" />
6466
</a>
65-
<a href="https://www.linkedin.com/company/openforge/" id="linkedin" target="_blank" rel="noopener">
67+
<a href="https://www.linkedin.com/company/openforge/" id="linkedin" target="_blank" rel="noopener" aria-label="Openforge Linkedin account link">
6668
<i class="fab fa-linkedin" aria-hidden="true" />
6769
</a>
68-
<a href="https://www.facebook.com/openforgemobile/" id="facebook" target="_blank" rel="noopener">
70+
<a href="https://www.facebook.com/openforgemobile/" id="facebook" target="_blank" rel="noopener" aria-label="Openforge Facebook account link">
6971
<i class="fab fa-facebook-square" aria-hidden="true" />
7072
</a>
7173
</p>

0 commit comments

Comments
 (0)