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

Unified Diff: includes/sidebar/secondary-navigation.tmpl

Issue 29399777: Issue 4976 - remove language selector AND noissue - fix sidebar offset (Closed) Base URL: https://hg.adblockplus.org/web.acceptableads.com
Patch Set: Created March 31, 2017, 3: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 | templates/default.tmpl » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: includes/sidebar/secondary-navigation.tmpl
===================================================================
--- a/includes/sidebar/secondary-navigation.tmpl
+++ b/includes/sidebar/secondary-navigation.tmpl
@@ -18,21 +18,9 @@
<ul>
<li>
<a id="contact-envelope" href="mailto:info@eyeo.com">
<span class="sr-only">{{ "Contact" | translate("sidebar-contact-label", "Contact label")}}</span>
<img src="/img/png/icon-contact.png" srcset="/img/png/icon-contact.png, /img/svg/icon-contact.svg 2x" aria-hidden="true">
</a>
</li>
- <li>
- <span class="sr-only">{{ "Select language" | translate("sidebar-language-label", "Language select label") }}</span>
- <select id="language-select" name="lang">
saroyanm 2017/03/31 15:16:28 We can delete all styles related to the "#language
- {% for lang in available_locales %}
- <option
- {% if locale == lang %} selected="selected" {% endif %}
- value="{{ source.resolve_link(page, lang)[1] }}">
- {{ config.get("langnames", lang) }}
- </option>
- {% endfor %}
- </select>
- </li>
</ul>
« no previous file with comments | « no previous file | templates/default.tmpl » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld