Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: includes/footer.tmpl

Issue 29812593: Fixes #99 - Added social media anchor titles, images alt text, and header to footer locale file (Closed) Base URL: https://hg.adblockplus.org/web.adblockplus.org
Patch Set: Created June 21, 2018, 12:11 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | locales/en/footer.json » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: includes/footer.tmpl
===================================================================
--- a/includes/footer.tmpl
+++ b/includes/footer.tmpl
@@ -24,44 +24,64 @@
<h5>{{get_string("development", "menu")}}</h5>
<ul>
{{ pageitem("source") }}
<li><a href="https://issues.adblockplus.org/report/13" hreflang="en">{{ get_string("roadmap", "menu") }}</a></li>
{{ pageitem("tools") }}
<li><a href="https://eyeo.com/jobs/" hreflang="en">{{ get_string("jobs", "menu") }}</a></li>
</ul>
</nav>
+
<nav id="social-list" class="column one-fourth">
- <h5>{{get_string("follow-us", "menu")}}</h5>
+ <h5>{{get_string("follow-us-header", "footer")}}</h5>
<ul>
<li>
- <a href="https://twitter.com/AdblockPlus" target="_blank">
+ <a
+ href="https://twitter.com/AdblockPlus"
+ title="{{ get_string("twitter-title", "footer") }}"
+ target="_blank">
<img
src="/img/footer-twitter-glyphicon.png"
- alt="{{ "Twitter" | translate("twitter-alt", "Twitter glyphicon alt text")}}">
+ alt="{{ get_string("twitter-alt", "footer") }}">
</a>
</li>
<li>
- <a href="https://www.youtube.com/user/AdblockPlusOfficial" target="_blank">
- <img src="/img/footer-youtube-glyphicon.png">
+ <a
+ href="https://www.youtube.com/user/AdblockPlusOfficial"
+ title="{{ get_string("youtube-title", "footer") }}"
+ target="_blank">
+ <img
+ src="/img/footer-youtube-glyphicon.png"
+ alt="{{ get_string("youtube-alt", "footer") }}">
</a>
</li>
<li>
- <a href="https://www.facebook.com/adblockplus" target="_blank">
- <img src="/img/footer-facebook-glyphicon.png">
+ <a
+ href="https://www.facebook.com/adblockplus"
+ title="{{ get_string("facebook-title", "footer") }}"
+ target="_blank">
+ <img
+ src="/img/footer-facebook-glyphicon.png"
+ alt="{{ get_string("facebook-alt", "footer") }}">
</a>
</li>
<li>
- <a href="https://www.instagram.com/adblockplus/" target="_blank">
- <img src="/img/footer-instagram-glyphicon.png">
+ <a
+ href="https://www.instagram.com/adblockplus/"
+ title="{{ get_string("instagram-title", "footer") }}"
+ target="_blank">
+ <img
+ src="/img/footer-instagram-glyphicon.png"
+ alt="{{ get_string("instagram-alt", "footer") }}">
</a>
</li>
</ul>
</nav>
</div> {# .row #}
+
<div id="footer-legal">
<p id="copyright-notice">{{ get_string("copyright", "footer") }}</p>
<ul id="legal-list">
{% for pagename in ["terms", "privacy", "impressum"] %}
<li>{{ pagename | linkify }}{{ get_string(pagename, "menu") }}</a></li>
{% endfor %}
</ul>
</div> {# #footer-legal #}
« no previous file with comments | « no previous file | locales/en/footer.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld