agaric/patternlibrary
10
1
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Add title explaining back-to-top

Browse source 
I don't think that having an aria label is an overuse of aria; it might not be necessary but it makes
certain screenreaders don't try to read anything about the icon markup inside the link.
This commit is contained in:
benjamin melançon 2018-12-28 14:12:40 -05:00
parent d446e8cf5b
commit 360f7be9f4
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
agaric/templates/layout/region--footer.html.twig
View file

@ -9,7 +9,7 @@
{% if content %} {% if content %}
<div{{ attributes }}> <div{{ attributes }}>
<div class="column footer-logo is-half"> <div class="column footer-logo is-half">
<a href="#"><span class="icon"><i class="agc agc-logo"></i></span></a> <a href="#" title="Back to top" aria-label="Back to top"><span class="icon"><i class="agc agc-logo"></i></span></a>
</div> </div>
<div class="column"> <div class="column">
Agaric LLC<br>Boston, MA 02134 Agaric LLC<br>Boston, MA 02134