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

Unified Diff: templates/default.tmpl

Issue 29551738: Issue 5634 - Replaced logo and refactored navbar width and colors (Closed) Base URL: https://hg.adblockplus.org/web.adblockplus.org
Patch Set: Rebased Created Sept. 25, 2017, 3:10 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 | « static/img/header-logo-2x.png ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: templates/default.tmpl
===================================================================
--- a/templates/default.tmpl
+++ b/templates/default.tmpl
@@ -73,40 +73,45 @@
<body>
<noscript>
<link rel="stylesheet" href="/css/noscript-desktop.css" media="(min-width: 1000px)"/>
<link rel="stylesheet" href="/css/noscript-mobile.css" media="(max-width: 1000px)"/>
</noscript>
{% macro pageitem(name) %}
{% if name == page %}
- <li class="selected">{{get_string(name, "menu")}}</li>
+ <li class="selected"><a>{{get_string(name, "menu")}}</a></li>
{% else %}
<li>{{name|linkify}}{{get_string(name, "menu")}}</a></li>
{% endif %}
{% endmacro %}
- <header>
- {{"index"|linkify(id="logo", class="sprite", itemprop="image")}}</a>
- <nav>
- <ul>
+ <nav class="navbar accent">
+ <div class="container">
+ {{ "index" | linkify(class="navbar-brand")}}
+ <img
+ alt="Adblock Plus"
+ title="Adblock Plus"
+ src="/img/header-logo.png"
+ srcset="/img/header-logo-2x.png">
+ </a>
+ <ul class="navbar-nav">
{% if localefile == "index" %}
- <li class="selected first">{{get_string("installation", "menu")}}</li>
+ <li class="selected first"><a>{{get_string("installation", "menu")}}</a></li>
{% else %}
<li class="install-link first">{{"index"|linkify}}{{get_string("installation", "menu")}} <span class="sprite install-link-icon"></span></a></li>
{% endif %}
{% for name in ["about", "features", "bugs", "contribute"] %}
{{pageitem(name)}}
{% endfor %}
- <li id="language">
- <div id="current-language">
+ <li id="language" class="dropdown">
+ <a href="#" id="current-language" class="dropdown-toggle">
{{ config.get("langnames", locale) }} ({{ locale | upper }})
- <span id="language-arrow" class="sprite"></span>
- </div>
- <ul id="language-selector">
+ </a>
+ <ul id="language-selector" class="dropdown-menu secondary">
{% for available_locale in available_locales %}
<li class="language-entry">
{{ page | linkify(available_locale) }}
{{ config.get("langnames", available_locale) }} ({{ available_locale | upper }})
</a>
</li>
{% endfor %}
</ul>
« no previous file with comments | « static/img/header-logo-2x.png ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld