From f4932816fd51a8ad0082dbacc305ce0811a66f34 Mon Sep 17 00:00:00 2001
From: Daniel Goldsmith <dgold@ascraeus.org>
Date: Wed, 31 Jul 2013 23:42:42 +0100
Subject: [PATCH 1/3] Changed tag_cloud documentation to produce a working tag
 cloud

---
 docs/settings.rst | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/docs/settings.rst b/docs/settings.rst
index eb0d028f..95f87e0e 100644
--- a/docs/settings.rst
+++ b/docs/settings.rst
@@ -497,12 +497,22 @@ The default theme does not include a tag cloud, but it is pretty easy to add::
 
     <ul>
         {% for tag in tag_cloud %}
-            <li class="tag-{{ tag.1 }}"><a href="{{ SITEURL }}/{{ tag.0.url }}">{{ tag.0 }}</a></li>
+            <span class="tag-{{ tag.1 }}"><a href="{{ SITEURL }}/{{ tag.0.url }}">{{ tag.0 }}</a>&nbsp;</span>
         {% endfor %}
     </ul>
 
 You should then also define a CSS style with the appropriate classes (tag-0 to tag-N, where
-N matches `TAG_CLOUD_STEPS` -1).
+N matches `TAG_CLOUD_STEPS` -1), tag-0 being the largest.
+
+      span.tag-0 {
+      font-size: 150%;
+      }
+
+      span.tag-1 {
+      font-size: 120%;
+      }
+
+      ...    
 
 Translations
 ============

From 20767ac73748516ee083f8af4972bd6ad83c4888 Mon Sep 17 00:00:00 2001
From: daniel goldsmith <dgold@ascraeus.org>
Date: Thu, 1 Aug 2013 11:34:44 +0100
Subject: [PATCH 2/3] deleted ul element from tag cloud

---
 docs/settings.rst | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/docs/settings.rst b/docs/settings.rst
index 95f87e0e..3be6621d 100644
--- a/docs/settings.rst
+++ b/docs/settings.rst
@@ -495,11 +495,11 @@ Setting name (default value)                        What does it do?
 
 The default theme does not include a tag cloud, but it is pretty easy to add::
 
-    <ul>
+    <p>
         {% for tag in tag_cloud %}
             <span class="tag-{{ tag.1 }}"><a href="{{ SITEURL }}/{{ tag.0.url }}">{{ tag.0 }}</a>&nbsp;</span>
         {% endfor %}
-    </ul>
+    </p>
 
 You should then also define a CSS style with the appropriate classes (tag-0 to tag-N, where
 N matches `TAG_CLOUD_STEPS` -1), tag-0 being the largest.

From 28ed75778ada23fa3130a6a0cc4d841288b56d55 Mon Sep 17 00:00:00 2001
From: Daniel Goldsmith <dgold@ascraeus.org>
Date: Tue, 6 Aug 2013 13:17:25 +0100
Subject: [PATCH 3/3] Made tag_cloud a list as per apparent consensus.

---
 docs/settings.rst | 32 +++++++++++++++++++++-----------
 1 file changed, 21 insertions(+), 11 deletions(-)

diff --git a/docs/settings.rst b/docs/settings.rst
index 44585b32..aafeb22e 100644
--- a/docs/settings.rst
+++ b/docs/settings.rst
@@ -521,22 +521,32 @@ Setting name (default value)                        What does it do?
 
 The default theme does not include a tag cloud, but it is pretty easy to add::
 
-    <p>
+    <ul class="tagcloud">
         {% for tag in tag_cloud %}
-            <span class="tag-{{ tag.1 }}"><a href="{{ SITEURL }}/{{ tag.0.url }}">{{ tag.0 }}</a>&nbsp;</span>
+            <li class="tag-{{ tag.1 }}"><a href="{{ SITEURL }}/{{ tag.0.url }}">{{ tag.0 }}</a></li>
         {% endfor %}
-    </p>
+    </ul>
 
-You should then also define a CSS style with the appropriate classes (tag-0 to tag-N, where
-N matches `TAG_CLOUD_STEPS` -1), tag-0 being the largest.
+You should then also define CSS styles with appropriate classes (tag-0 to tag-N, where
+N matches `TAG_CLOUD_STEPS` -1), tag-0 being the most frequent, and define a ul.tagcloud 
+class with appropriate list-style to create the cloud, for example::
 
-      span.tag-0 {
-      font-size: 150%;
-      }
+    ul.tagcloud {
+    	list-style: none;
+        padding: 0;
+    }
 
-      span.tag-1 {
-      font-size: 120%;
-      }
+    ul.tagcloud li {
+        display: inline-block;
+    }
+
+    li.tag-0 {
+        font-size: 150%;
+    }
+
+    li.tag-1 {
+        font-size: 120%;
+    }
 
       ...