Skip to content

Commit 82153eb

Browse files
committed
Adds aria labels for links
1 parent d542034 commit 82153eb

File tree

3 files changed

+6
-6
lines changed

3 files changed

+6
-6
lines changed

app/_includes/header-footer/navigation.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
<nav class="navigation">
22
<h2>Navigation</h2>
3-
<a id="menu-button" href="#">
3+
<a id="menu-button" href="#" aria-hidden="true">
44
<i class="fa fa-bars"></i>
55
<i class="fa fa-times"></i>
66
</a>

app/_includes/listing/person.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,10 @@
33
<figure>
44
{% if group_members.size == 2 or group_members.size == 3 %}
55
{% for member in group_members %}
6-
<a href="{{ member.url }}"><img src="/images/people/{{ member.photo }}" /></a>
6+
<a href="{{ member.url }}" aria-label="Profile image of {{ person.name }}"><img src="/images/people/{{ member.photo }}" aria-hidden="true" /></a>
77
{% endfor %}
88
{% else %}
9-
<a href="{{ person.url }}"><img src="/images/people/{{ person.photo }}" /></a>
9+
<a href="{{ person.url }}" aria-label="Profile image of {{ person.name }}"><img src="/images/people/{{ person.photo }}" aria-hidden="true" /></a>
1010
{% endif %}
1111
</figure>
1212
</div>
Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
11
<ul class="social-links new-window">
22
<li>
3-
<a href="http://twitter.com/share?text={{ include.post.title | uri_escape }}&amp;url=http://{{ site.name }}{{ include.post.url | uri_escape }}&amp;via=tw_arts"><i class="fab fa-twitter"></i></a>
3+
<a href="https://twitter.com/share?text={{ include.post.title | uri_escape }}&amp;url=https://{{ site.name }}{{ include.post.url | uri_escape }}&amp;via=tw_arts" aria-label="Twitter"><i class="fab fa-twitter" aria-hidden="true"></i></a>
44
</li>
55
<li>
6-
<a href="http://www.facebook.com/sharer/sharer.php?title={{ include.post.title | uri_escape }}&amp;u=http://{{ site.name }}{{ include.post.url | uri_escape }}"><i class="fab fa-facebook"></i></a>
6+
<a href="https://www.facebook.com/sharer/sharer.php?title={{ include.post.title | uri_escape }}&amp;u=https://{{ site.name }}{{ include.post.url | uri_escape }}" aria-label="Facebook"><i class="fab fa-facebook" aria-hidden="true"></i></a>
77
</li>
88
<li>
9-
<a href="http://www.tumblr.com/share?v=3&amp;t={{ include.post.title | uri_escape }}&amp;u=http://{{ site.name }}{{ include.post.url | uri_escape }}"><i class="fab fa-tumblr"></i></a>
9+
<a href="https://www.tumblr.com/share?v=3&amp;t={{ include.post.title | uri_escape }}&amp;u=https://{{ site.name }}{{ include.post.url | uri_escape }}" aria-label="Tumblr"><i class="fab fa-tumblr" aria-hidden="true"></i></a>
1010
</li>
1111
</ul>

0 commit comments

Comments
 (0)