Fix: use relative links for menu items when possible (mmistakes#1783) (…

…mmistakes#1784)
jchwenger · Aug 13, 2018 · 0a60ec7 · 0a60ec7
1 parent ca6612d
commit 0a60ec7
Showing 1 changed file with 9 additions and 9 deletions.
diff --git a/_includes/masthead.html b/_includes/masthead.html
@@ -4,16 +4,16 @@
       <nav id="site-nav" class="greedy-nav">
         <a class="site-title" href="{{ '/' | relative_url }}">{{ site.title }}</a>
         <ul class="visible-links">
-          {% for link in site.data.navigation.main %}
-            {% if link.url contains 'http' %}
-              {% assign domain = '' %}
-            {% else %}
-              {% assign domain = site.url | append: site.baseurl %}
-            {% endif %}
+          {%- for link in site.data.navigation.main -%}
+            {%- if link.url contains '://' -%}
+              {%- assign url = link.url -%}
+            {%- else -%}
+              {%- assign url = link.url | relative_url -%}
+            {%- endif -%}
             <li class="masthead__menu-item">
-              <a href="{{ domain }}{{ link.url }}" {% if link.description %}title="{{ link.description }}"{% endif %}>{{ link.title }}</a>
+              <a href="{{ url }}" {% if link.description %}title="{{ link.description }}"{% endif %}>{{ link.title }}</a>
             </li>
-          {% endfor %}
+          {%- endfor -%}
         </ul>
         {% if site.search == true %}
         <button class="search__toggle" type="button">
@@ -30,4 +30,4 @@
       </nav>
     </div>
   </div>
-</div>
+</div>