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

Unified Diff: includes/navbar.tmpl

Issue 29689571: Fixes #4 - Update primary and footer navigation contents (Closed) Base URL: https://hg.adblockplus.org/web.adblockplus.org
Patch Set: Created Feb. 5, 2018, 9:43 a.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 | « includes/footer.tmpl ('k') | locales/en/menu.json » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: includes/navbar.tmpl
===================================================================
--- a/includes/navbar.tmpl
+++ b/includes/navbar.tmpl
@@ -9,21 +9,24 @@
<span>{{ "Adblock <strong>Plus</strong>" | translate("navbar-logo-text", "Navbar logo text") }}</span>
</a>
<a href="#" id="navbar-menu-toggle">
<img
src="/img/menu-toggle.png"
srcset="/img/menu-toggle.svg 2x">
</a>
<ul id="navbar-menu">
- {% for pagename in ["about", "features", "bugs", "contribute"] %}
saroyanm 2018/02/05 15:44:10 In the Issue it says: "Features are getting merged
saroyanm 2018/02/05 15:44:10 What was the reason of making the Issue reporting
ire 2018/02/05 16:15:01 Yes, this change will not be pushed to master unti
ire 2018/02/05 16:15:02 Yes it's been through the QA team. I don't know sp
+ {% for pagename in ["about", "contribute"] %}
<li>
{{ pagename | linkify }}{{ get_string(pagename, "menu") }}</a>
</li>
{% endfor %}
+ <li>
+ {{ "documentation" | linkify }}{{ get_string("help", "menu") }}</a>
+ </li>
<li id="navbar-locale-menubar">
<a href="#" id="navbar-locale-selected">
{{ config.get("langnames", locale) }} ({{ locale | to_og_locale | to_og_location }})
</a>
<ul id="navbar-locale-menu">
{% for available_locale in available_locales %}
<li>
{{ page | linkify(available_locale) }}
« no previous file with comments | « includes/footer.tmpl ('k') | locales/en/menu.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld