Performance tuning

This commit adds a lot of performance tuning, making the page compile
much faster (<= 10 seconds)

README.md

 # Repository holding the sources of the KWARC.info website
 
-**Building this page can take up to 30 seconds. This is normal and to be expected.**
-
 Note for authors: When editing some text, please do not hard wrap existing lines. 
 This plays very bad with diffs, in particular an entire paragraph shows up as changed even though it might just be a single line. 
 Furthermore, this introduces lots and lots of merge conflicts
@@ -25,6 +23,21 @@ See the above links for details.
 * `project/` -- people -- event_activites: folders used to generate menu pages (as set in `_config.yml`)
 * `_tagpages` : the tag pages, each tag gets an almost empty md file so that the tag page gets generated by jekyll
 
+## Performance
+This jekyll page is tuned in order to build as fast as possible. 
+It also makes use of the [jekyll-compress-html](https://github.com/penibelst/jekyll-compress-html) layout, to be efficiently transferable over network. 
+During the evolution of the website, the build time has changed dramatically. 
+It used to take around 5 minutes to build, with a few optimizations it now only takes up 15 seconds. 
+
+The biggest slowdowns were:
+    * unneeded iterations
+        * iterations within iterations (usually not required, if one thinks carefully)
+        * `if` conditions within a loop, instead of a `where` clause
+        * full iteration to extract a single item (use `first` instead)
+    * repeated sorting of `site.pages` by the same `menu_order` key (instead sort this once and use a global variable)
+    * multiple chained `if`s instead of a single `and`
+    * unneeded variable assignments (these seem to take a lot of time in liquid)
+
 ## How to use Jekyll to test/build this website
 
 This is a [*static website*](http://en.wikipedia.org/wiki/Static_web_page) automatically generated with [Jekyll](http://jekyllrb.com/) by [GitHub Pages](http://pages.github.com/).
@@ -91,10 +104,10 @@ Furthermore, if these are mixed between folders and static items, the folders wi
 - menu_title: 'FAU links'
   items:
     - menu_title: 'Overview'
-      menu_external: true
+      external: true
       url: 'https://fau.de/'
     - menu_title: 'CS department'
-      menu_external: true
+      external: true
       url: 'https://cs.fau.de/'
 ```
 

_config.yml

@@ -85,5 +85,5 @@ defaults:
       path: "" 
     values:
       menu_hidden: false
-      menu_external: false
       menu_order: 100
+      external: false
\ No newline at end of file

_includes/footer.html

@@ -13,10 +13,6 @@
         <a href="https://github.com/KWARC" title="GitHub" class="waves-effect waves-teal btn-flat">
           <i class="fa fa-github" aria-hidden="true"></i> GitHub
         </a>
-        
-        <!--
-        
-        -->
       
         <a href="{{ site.baseurl }}/atom.xml" title="Atom Feed" class="waves-effect waves-teal btn-flat">
           <i class="fa fa-rss" aria-hidden="true"></i> Atom Feed

_includes/head.html

@@ -16,7 +16,7 @@
     
     <!--Import Google Icon Font -->
     <link href="https://fonts.googleapis.com/icon?family=Material+Icons" rel="stylesheet">
-      
+    
     <!-- Import materialize -->
     <link type="text/css" rel="stylesheet" href="{{ site.baseurl }}/public/{{ site.materialize }}/css/materialize.min.css"  media="screen,projection"/>
     

_includes/header.html

@@ -3,7 +3,7 @@
   <div class="container">
     <div class="nav-wrapper">
       <a class="page-title">
-        {% if page.title == "Home" %}{{ site.title }}{% else %}{{ page.title }}{% endif %}
+        {% if page.url == "/" %}{{ site.title }}{% else %}{{ page.title }}{% endif %}
       </a>
     </div>
   </div>
@@ -23,10 +23,8 @@
     </div>
   </li>
   
-  {% for item in site.menu | sort:"menu_order %}
-    {% if item.folder %}
-      {% include menu_sub.html %}
-    {% elsif item.items %}
+  {% for item in site.menu | sort:"menu_order" %}
+    {% if (item.folder or item.items) %}
       {% include menu_sub.html %}
     {% else %}
       {% include menu_single.html %}

_includes/menu_single.html

@@ -6,10 +6,12 @@
   <li
     {% if page.url == item.url %}class="active"{% endif %}
   >
-    <a
-      href="{% unless item.menu_external %}{{ site.baseurl }}{% endunless %}{{ item.url }}"
-      {% if item.menu_external %}target="_blank"{% endif %}
-    >
+    {% if item.external %}
+        <a href="{{ item.url }}" target="_blank">
+    {% else %}
+        <a href="{{ site.baseurl }}{{ item.url }}">
+    {% endif %}
+    
       {% if item.menu_title %}
         {{ item.menu_title }}
       {% else %}

_includes/menu_sub.html

 {% comment %}
-  This page creates a submenu item. It expects an appropriate `item` as argument. 
+    This page creates a submenu item. It expects an appropriate `item` as argument. 
+    It also expects the global 'sorted_pages' argument. 
 {% endcomment %}
 
 {% unless item.menu_hidden %}
+    {% assign base_folder = page.url | split: '/' %}
 
-  <!-- Figure out our current location -->
-  {% assign url_parts = page.url | split: '/' %}
-  {% assign base_url = url_parts | last %}
-  {% assign base_folder = url_parts[1] %}
+    <!-- if we are in the right folder, we are active -->
+    {% if base_folder[1] == item.folder %}
+        {% assign menu_active = true %}
+    {% else %}
+        {% assign menu_active = false %}
+        {% for item in (item.items | where: "url", page.url) %}
+            {% unless item.external %}
+                {% assign menu_active = true %}
+                <!-- active because {{item}} -->
+            {% endunless %}
+        {% endfor %}
+    {% endif %}
 
-  <!-- Find out if we are active -->
-  {% assign menu_active = false %}
 
-  <!-- if we are in the right folder, we are active -->
-  {% if base_folder == item.folder %}
-    {% assign menu_active = true %}
-  {% endif %}
-  
-  <!-- check if we are in the current oder -->
-    {% for item in item.items %}
-        {% unless item.menu_page %}
-            {% unless item.external_page %}
-                {% if item.url == page.url %}
-                    {% assign menu_active = true %}
-                {% endif %}
-            {% endunless %}
-        {% endunless %}
-    {% endfor %}
-    
-  {% assign pages = site.pages | sort: "menu_order"%}
-  
-  <!-- Find the current folder -->
-  {% assign item_folder = item.folder %}
+    <!-- Find the current folder -->
+    {% assign item_folder = item.folder %}
 
-  <li class="no-padding">
-    <ul class="collapsible collapsible-accordion">
-      <li class="bold">
-        <a class="collapsible-header {% if menu_active %}active{% endif %}">
-        {% if item.menu_title %}
-          {{ item.menu_title }}
-        {% else %}
-          {{ item.title }}
-        {% endif %}
-        </a>
-        <div class="collapsible-body">
-          <ul>
-              <!-- If we have folders, iterate over them -->
-              {% if item.folder %}
-                {% for item in pages %}
-                    <!-- get the current page -->
-                    {% assign sub_parts = item.url | split: '/' %}
-                    {% assign sub_url = sub_parts | last %}
-                    {% assign sub_folder = sub_parts[1] %}
-                    
-                    <!-- if we are in the right folder, include it -->
-                    {% if sub_folder == item_folder %}
-                      {% include menu_single.html %}
+    <li class="no-padding">
+        <ul class="collapsible collapsible-accordion">
+            <li class="bold">
+                <a class="collapsible-header {% if menu_active %}active{% endif %}">
+                    {% if item.menu_title %}
+                        {{ item.menu_title }}
+                    {% else %}
+                        {{ item.title }}
                     {% endif %}
-                {% endfor %}
-              {% endif %}
-              
-              <!-- if we have items, iterate over them -->
-              {% if item.items %}
-                {% assign sitems = item.items | sort: "menu_order"%}
-                {% for item in sitems %}
-                  {% include menu_single.html %}
-                {% endfor %}
-              {% endif %}
-          </ul>
-        </div>
-      </li>
-    </ul>
-  </li>
+                </a>
+                <div class="collapsible-body">
+                    <ul>
+                        <!-- If we have folders, iterate over them -->
+                        {% if item.folder %}
+                            {% for item in sorted_pages %}
+                            
+                                {% assign sub_components = item.url | split: '/' %}
+                                {% if sub_components[1] == item_folder %}
+                                    {% include menu_single.html %}
+                                {% endif %}
+                            {% endfor %}
+                        {% endif %}
+
+                        <!-- if we have items, iterate over them -->
+                        {% if item.items %}
+                            {% for item in (item.items | sort: "menu_order") %}
+                                {% include menu_single.html %}
+                            {% endfor %}
+                        {% endif %}
+                    </ul>
+                </div>
+            </li>
+        </ul>
+    </li>
 {% endunless %}
\ No newline at end of file

_includes/people_chip.html

-{% capture purl %}/people/{{person}}/{% endcapture %}
-{% assign pname = person %}
-{% assign pimg = "" %}
+{% comment %}
+    This page creates a single 'chip' for linking to a person. 
+    It expects the username of the person as the 'person' variable
+{% endcomment %}
 
-{% for pp in site.pages %}
-  {% if pp.url == purl %}
-    {% assign pname = pp.fullname %}
-    {% assign pimg = pp.pic %}
-  {% endif %}
-{% endfor %}
+{% capture purl %}/people/{{person}}/{% endcapture %}
+{% assign pp = site.pages | where: "url", purl | first %}
 
-<a href="{{ site.baseurl }}{{purl}}">
-  <div class="chip">
-      {% if pimg != "" %}
-        <img src="{{ site.baseurl }}/{{pimg}}">
-      {% endif %}
-      {{pname}}
-  </div>
-</a>
+{% if pp %}
+    <a href="{{ site.baseurl }}{{purl}}">
+        <div class="chip">
+            <img src="{{ site.baseurl }}/{{pp.pic}}">
+            {{pp.fullname}}
+        </div>
+    </a>
+{% else %}
\ No newline at end of file
+    <a href="{{ site.baseurl }}{{purl}}">
+        <div class="chip">
+            {{person}}
+        </div>
+    </a>
+{% endif %}
\ No newline at end of file

_includes/post_link.html

+{% comment %}
+    Generates a link to a single post. 
+    Expects the page representing the post as a 'post' variable. 
+{% endcomment %}
 
 <div class="post-link">
     <h3 class="post-title">
@@ -7,8 +11,7 @@
     </h3>
     <p class="post-meta" >
         {% assign firsttag = post.tags | first %}
-        {% assign tmp_list = site.tagpages | where:"tag", firsttag %}
-        {% assign type = tmp_list | first %}
+        {% assign type = site.tagpages | where:"tag", firsttag | first %}
         <span class="post-type">{{ type.title }} by {{post.author}}</span> -
         {% if post.location %}<span class="post-location">{{post.location}}</span> - {% endif %}
         <span class="post-date">{{ post.date | date_to_string }}</span>

_includes/posts.html

----
-layout: page
-title: Project Activites
-alltags: True
-menu_order: 101
----
-
-{% comment %}
-Project activities are generated through the /_posts/ folder
-{% endcomment %}
-
-<h2>By tag</h2>
-
-{% assign with_post_numbers = true %}
-{% include tags_module.html %}
-
-<div class="tagcloud" >
-    <span class="tag" ><a class="btn btn-default" href="{{ site.blog_path}}">All activites <span class="badge">{{ site.posts.size }}</span></a></span>
-{{ tagscontent }}
-</div>
-
-
-<h2>Most recent activites</h2>
-
-{% for post in site.posts %}
-    {% if forloop.index < 5 %}
-    {% include post_link.html %}
-    {% endif %}
-{% endfor %}
-
-<a href="{{ site.blog_path}}">See all</a>
-
-

_layouts/default.html

 ---
 layout: compress
 ---
+{% assign sorted_pages = site.pages | sort: "menu_order" %}
 
 <!DOCTYPE html>
 <html lang="en-us">