Use a sr-only span rather than the title attribute to include text in…

… the footer social links. For #177.
middlebury · Jul 23, 2019 · 03ca44c · 03ca44c
1 parent c0ed0d1
commit 03ca44c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/src/templates/partials/footer.twig b/src/templates/partials/footer.twig
@@ -19,9 +19,9 @@
               <li class="social-links__item">
                 <a href="{{item.href}}" class="social-links__link">
                   {% include 'partials/icon.twig' with {
-                    'name': 'item.slug',
-                    'title': 'item.text'
+                    'name': 'item.slug'
                   } %}
+                  <span class="sr-only">{{item.text}}</span>
                 </a>
               </li>
             {% endfor %}

diff --git a/src/templates/partials/icon.twig b/src/templates/partials/icon.twig
@@ -1,4 +1,3 @@
 <svg class="icon {{class}} {% if size %}icon--{{size}}{% endif %}" focusable="false" aria-hidden="true">
-  {% if title %}<title>{{title}}</title>{% endif %}
   <use xlink:href="#icon-{{name}}">
 </svg>