From d52f6052005f4e4fa701e90b3b60f7ebf13fe035 Mon Sep 17 00:00:00 2001
From: squidfunk <martin.donath@squidfunk.com>
Date: Sun, 26 Feb 2017 15:06:50 +0100
Subject: [PATCH] Refactored and cleaned up partials

---
 material/partials/header.html    | 10 ++++----
 material/partials/tabs-item.html | 24 +++++++++++++++-----
 material/partials/tabs.html      |  6 ++++-
 src/base.html                    |  4 ++--
 src/partials/header.html         | 10 ++++----
 src/partials/nav.html            |  6 +++++
 src/partials/tabs-item.html      | 39 +++++++++++++++++++-------------
 src/partials/tabs.html           | 14 +++++++-----
 8 files changed, 72 insertions(+), 41 deletions(-)

diff --git a/material/partials/header.html b/material/partials/header.html
index 39e85d2ca..fda797783 100644
--- a/material/partials/header.html
+++ b/material/partials/header.html
@@ -35,11 +35,11 @@
         {% endblock %}
       </div>
       <div class="md-flex__cell md-flex__cell--shrink">
-          <div class="md-header-nav__source">
-            {% if config.repo_url %}
-              {% include "partials/source.html" %}
-            {% endif %}
-          </div>
+        <div class="md-header-nav__source">
+          {% if config.repo_url %}
+            {% include "partials/source.html" %}
+          {% endif %}
+        </div>
       </div>
     </div>
   </nav>
diff --git a/material/partials/tabs-item.html b/material/partials/tabs-item.html
index 6649bbe94..aef42d77e 100644
--- a/material/partials/tabs-item.html
+++ b/material/partials/tabs-item.html
@@ -1,13 +1,25 @@
 {% if nav_item.is_homepage %}
   <li class="md-tabs__item">
-    <a href="{{ nav_item.url }}" title="{{ nav_item.title }}" class="md-tabs__link {% for nav_item_ in nav %} {% if nav_item_.url == page.url %} md-tabs__link--active {% endif %} {% endfor %}">
-      {{ nav_item.title }}
-    </a>
+    {% if not page.ancestors | length and nav | selectattr("url", page.url) %}
+      <a href="{{ nav_item.url }}" title="{{ nav_item.title }}" class="md-tabs__link md-tabs__link--active">
+        {{ nav_item.title }}
+      </a>
+    {% else %}
+      <a href="{{ nav_item.url }}" title="{{ nav_item.title }}" class="md-tabs__link">
+        {{ nav_item.title }}
+      </a>
+    {% endif %}
   </li>
 {% elif nav_item.children and nav_item.children | length > 0 %}
   <li class="md-tabs__item">
-    <a href="{{ (nav_item.children | first).url }}" title="{{ nav_item.title }}" class="md-tabs__link {% if nav_item.active %} md-tabs__link--active {% endif %}">
-      {{ nav_item.title }}
-    </a>
+    {% if nav_item.active %}
+      <a href="{{ (nav_item.children | first).url }}" title="{{ nav_item.title }}" class="md-tabs__link md-tabs__link--active">
+        {{ nav_item.title }}
+      </a>
+    {% else %}
+      <a href="{{ (nav_item.children | first).url }}" title="{{ nav_item.title }}" class="md-tabs__link">
+        {{ nav_item.title }}
+      </a>
+    {% endif %}
   </li>
 {% endif %}
diff --git a/material/partials/tabs.html b/material/partials/tabs.html
index 3ed208f82..e040436bf 100644
--- a/material/partials/tabs.html
+++ b/material/partials/tabs.html
@@ -1,4 +1,8 @@
-<nav class="md-tabs {% if page.ancestors | length > 0 %} md-tabs--active {% endif %}" data-md-component="tabs">
+{% set class = "md-tabs" %}
+{% if page.ancestors | length > 0 %}
+  {% set class = "md-tabs md-tabs--active" %}
+{% endif %}
+<nav class="{{ class }}" data-md-component="tabs">
   <div class="md-tabs__inner md-grid">
     <ul class="md-tabs__list">
       {% for nav_item in nav %}
diff --git a/src/base.html b/src/base.html
index 58e6c3990..22cd83b6d 100644
--- a/src/base.html
+++ b/src/base.html
@@ -57,7 +57,7 @@
 
       <!-- Generator banner -->
       <meta name="generator"
-        content="mkdocs-{{ mkdocs_version }}, $theme-name$-$theme-version$" />
+          content="mkdocs-{{ mkdocs_version }}, $theme-name$-$theme-version$" />
     {% endblock %}
 
     <!-- Block: site title -->
@@ -96,7 +96,7 @@
                     | default("Roboto Mono") %}
         {% set font = text + ':300,400,400i,700|' + code | replace(' ', '+') %}
         <link rel="stylesheet" type="text/css"
-          href="https://fonts.googleapis.com/css?family={{ font }}" />
+            href="https://fonts.googleapis.com/css?family={{ font }}" />
         <style>
           body, input {
             font-family: "{{ text }}", "Helvetica Neue",
diff --git a/src/partials/header.html b/src/partials/header.html
index 16f9877c2..a472b1608 100644
--- a/src/partials/header.html
+++ b/src/partials/header.html
@@ -77,11 +77,11 @@
 
       <!-- Repository containing source -->
       <div class="md-flex__cell md-flex__cell--shrink">
-          <div class="md-header-nav__source">
-            {% if config.repo_url %}
-              {% include "partials/source.html" %}
-            {% endif %}
-          </div>
+        <div class="md-header-nav__source">
+          {% if config.repo_url %}
+            {% include "partials/source.html" %}
+          {% endif %}
+        </div>
       </div>
     </div>
   </nav>
diff --git a/src/partials/nav.html b/src/partials/nav.html
index abbc18cd7..544cf1905 100644
--- a/src/partials/nav.html
+++ b/src/partials/nav.html
@@ -22,6 +22,8 @@
 
 <!-- Main navigation -->
 <nav class="md-nav md-nav--primary" data-md-level="0">
+
+  <!-- Site title -->
   <label class="md-nav__title md-nav__title--site" for="drawer">
     {% if config.extra.logo %}
       <i class="md-logo md-nav__button">
@@ -32,11 +34,15 @@
     {% endif %}
     {{ config.site_name }}
   </label>
+
+  <!-- Repository containing source -->
   {% if config.repo_url %}
     <div class="md-nav__source">
       {% include "partials/source.html" %}
     </div>
   {% endif %}
+
+  <!-- Render item list -->
   <ul class="md-nav__list" data-md-scrollfix>
     {% for nav_item in nav %}
       {% set path = "nav-" + loop.index | string %}
diff --git a/src/partials/tabs-item.html b/src/partials/tabs-item.html
index 26bccc7a5..acfb04ede 100644
--- a/src/partials/tabs-item.html
+++ b/src/partials/tabs-item.html
@@ -23,26 +23,33 @@
 <!-- Home page -->
 {% if nav_item.is_homepage %}
   <li class="md-tabs__item">
-    <a href="{{ nav_item.url }}" title="{{ nav_item.title }}"
-        class="md-tabs__link
-          {% for nav_item_ in nav %}
-            {% if nav_item_.url == page.url %}
-              md-tabs__link--active
-            {% endif %}
-          {% endfor %}">
-      {{ nav_item.title }}
-    </a>
+    {% if not page.ancestors | length and nav | selectattr("url", page.url) %}
+      <a href="{{ nav_item.url }}" title="{{ nav_item.title }}"
+          class="md-tabs__link md-tabs__link--active">
+        {{ nav_item.title }}
+      </a>
+    {% else %}
+      <a href="{{ nav_item.url }}" title="{{ nav_item.title }}"
+          class="md-tabs__link">
+        {{ nav_item.title }}
+      </a>
+    {% endif %}
   </li>
 
 <!-- Main navigation item with nested items -->
 {% elif nav_item.children and nav_item.children | length > 0 %}
   <li class="md-tabs__item">
-    <a href="{{ (nav_item.children | first).url }}"
-        title="{{ nav_item.title }}" class="md-tabs__link
-          {% if nav_item.active %}
-            md-tabs__link--active
-          {% endif %}">
-      {{ nav_item.title }}
-    </a>
+    {% if nav_item.active %}
+      <a href="{{ (nav_item.children | first).url }}"
+          title="{{ nav_item.title }}"
+          class="md-tabs__link md-tabs__link--active">
+        {{ nav_item.title }}
+      </a>
+    {% else %}
+      <a href="{{ (nav_item.children | first).url }}"
+          title="{{ nav_item.title }}" class="md-tabs__link">
+        {{ nav_item.title }}
+      </a>
+    {% endif %}
   </li>
 {% endif %}
diff --git a/src/partials/tabs.html b/src/partials/tabs.html
index ced9c20be..8efd7949b 100644
--- a/src/partials/tabs.html
+++ b/src/partials/tabs.html
@@ -20,12 +20,14 @@
   IN THE SOFTWARE.
 -->
 
-<!-- Tabs with outline (level denotes the active level) -->
-<nav class="md-tabs
-      {% if page.ancestors | length > 0 %}
-        md-tabs--active
-      {% endif %}"
-    data-md-component="tabs">
+<!-- Determine class according to level -->
+{% set class = "md-tabs" %}
+{% if page.ancestors | length > 0 %}
+  {% set class = "md-tabs md-tabs--active" %}
+{% endif %}
+
+<!-- Tabs with outline -->
+<nav class="{{ class }}" data-md-component="tabs">
   <div class="md-tabs__inner md-grid">
     <ul class="md-tabs__list">
       {% for nav_item in nav %}