From 41bfe68edd063bdbba5bbaa0daa8fea020a8e7db Mon Sep 17 00:00:00 2001
From: Joseph Nix <nixjdm@terminallabs.com>
Date: Sat, 12 May 2018 11:59:57 -0500
Subject: [PATCH] [pluginlist-update] Displaying all lists of plugins in the
 same way, using the same macro. This caused an increase in build time, but
 has a better finished product.

---
 content/plugins/contents.lr    |  6 +++---
 templates/macros/plugins.html  |  4 ++--
 templates/plugin-category.html |  2 +-
 templates/plugin-tag.html      | 11 ++++-------
 templates/plugins.html         |  2 +-
 5 files changed, 11 insertions(+), 14 deletions(-)

diff --git a/content/plugins/contents.lr b/content/plugins/contents.lr
index 3be5ac2f..1dd1f3e9 100644
--- a/content/plugins/contents.lr
+++ b/content/plugins/contents.lr
@@ -7,10 +7,10 @@ body:
 #### text-block ####
 text:
 
-Lektor is a young project but it has a growing list of both official and community supported plugins. Official plugins were developed by the authors of Lektor and kept in good shape together with the rest of the project. Community plugins are moderated and updated regularly, but they might not keep pace with development on Lektor. They are maintained by the community. Install them with `lektor plugins add NAME`.
-
-This is a list of currently known plugins. Official plugins are prefixed with as asterisk (*) and are listed first in each section and separated by an underline.
+Lektor is a young project but it has a growing list of both official and community supported plugins. Official plugins are maintained by Lektor and kept in good shape together with the rest of the project. Community plugins are moderated and updated regularly, but they might not keep pace with development on Lektor. They are maintained by the community. Install them with `lektor plugins add NAME`.
 
 If you want to make or publish your own plugin, or see our [docs :ref](/docs/plugins).
+
+(*) Asterisks denote official plugins.
 ----
 class: default
diff --git a/templates/macros/plugins.html b/templates/macros/plugins.html
index bd57442c..17f9b21c 100644
--- a/templates/macros/plugins.html
+++ b/templates/macros/plugins.html
@@ -7,9 +7,9 @@
   </ul>
 {% endmacro %}
 
-{% macro render_plugin_list(category) %}
+{% macro render_plugin_list(query) %}
   <ul>
-    {% for plugin in category.children.order_by('-official', 'name') %}
+    {% for plugin in query.order_by('-official', 'name') %}
       {% set pd =  project_data(plugin.name) %}
       {% if plugin.summary %}
         {% set summary = plugin.summary %}
diff --git a/templates/plugin-category.html b/templates/plugin-category.html
index 7f02c9c1..de51b739 100644
--- a/templates/plugin-category.html
+++ b/templates/plugin-category.html
@@ -4,5 +4,5 @@
   <h1>{{ this.title }}</h1>
   {% call render_full_width_blocks(this.body.blocks, classes='col-md-12') %}
   {% endcall %}
-  {{ render_plugin_list(this) }}
+  {{ render_plugin_list(this.children) }}
 {% endblock %}
diff --git a/templates/plugin-tag.html b/templates/plugin-tag.html
index 4dfd38d0..fcaedde0 100644
--- a/templates/plugin-tag.html
+++ b/templates/plugin-tag.html
@@ -1,4 +1,5 @@
 {% extends "plugins.html" %}
+{% from "macros/plugins.html" import render_plugin_list %}
 
 {% block body %}
   <h1>Tag: {{ this.tag }}</h1>
@@ -23,11 +24,7 @@
   {% endfor %}
 
   <h2>Plugins:</h2>
-  <ul>
-    {% for plugin in this.items %}
-      <li>
-        <a href="{{ plugin|url }}">{{ plugin._id }}</a>
-      </li>
-    {% endfor %}
-  </ul>
+    <div class="plugin">
+      {{ render_plugin_list(this.items) }}
+    </div>
 {% endblock %}
diff --git a/templates/plugins.html b/templates/plugins.html
index 236746d3..83c489f4 100644
--- a/templates/plugins.html
+++ b/templates/plugins.html
@@ -18,7 +18,7 @@
   {% for category in site.query('/plugin-categories').order_by('title') %}
     <h2><a href="{{ category|url }}">{{ category.title }}</a></h2>
     <div class="plugin">
-      {{ render_plugin_list(category) }}
+      {{ render_plugin_list(category.children) }}
     </div>
   {% endfor %}
 {% endblock %}