Use up/down arrows as sorting indicator

.. rather than a red heading, which implies an error.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

5 files changed, 25 insertions, 6 deletions

diff --git a/htdocs/css/style.css b/htdocs/css/style.css
index 3b10edb..3604148 100644
--- a/htdocs/css/style.css
+++ b/htdocs/css/style.css
@@ -124,6 +124,10 @@ table.patchlist th {
 	padding-left: 6px;
 }
 
+table.patchlist th img {
+	vertical-align: bottom;
+}
+
 table.patchlist td {
 	padding: 2px 6px 2px 6px;
 	margin: 0px;
@@ -145,14 +149,11 @@ table.patchlist tr.even {
 	background: #e8e8e8;
 }
 
-a.colactive {
-	color: red;
-}
-
-a.colinactive {
+a.colinactive, a.colactive {
 	color: black;
 	text-decoration: none;
 }
+
 a.colinactive:hover {
 	color: red;
 }
diff --git a/htdocs/images/16-arrow-down.png b/htdocs/images/16-arrow-down.png
new file mode 100644
index 0000000..fa0ff7b
--- /dev/null
+++ b/htdocs/images/16-arrow-down.png
diff --git a/htdocs/images/16-arrow-up.png b/htdocs/images/16-arrow-up.png
new file mode 100644
index 0000000..60c4eb9
--- /dev/null
+++ b/htdocs/images/16-arrow-up.png
diff --git a/templates/patchwork/help/about.html b/templates/patchwork/help/about.html
index fa5094f..4cae691 100644
--- a/templates/patchwork/help/about.html
+++ b/templates/patchwork/help/about.html
@@ -15,5 +15,7 @@ web framework.</p>
 href="http://code.google.com/p/django-registration/">django-registration</a>
 application.</a>
 
+<p>Icons from the <a href="http://sweetie.sublink.ca/">Sweetie</a> icon set.</a>
+
 {% endblock %}
 
diff --git a/templates/patchwork/patch-list.html b/templates/patchwork/patch-list.html
index a0a8cec..57491e2 100644
--- a/templates/patchwork/patch-list.html
+++ b/templates/patchwork/patch-list.html
@@ -31,6 +31,10 @@
   <th>
    {% ifequal order.name "name" %}
     <a class="colactive"
+     href="{% listurl order=order.reversed_name %}"><img
+     src="/images/16-arrow-{% if order.reversed %}up{% else %}down{%endif%}.png"
+     width="16" height="16"
+    ></a> <a class="colactive"
      href="{% listurl order=order.reversed_name %}">Patch</a>
    {% else %}
     <a class="colinactive" href="{% listurl order="name" %}">Patch</a>
@@ -40,6 +44,10 @@
   <th>
    {% ifequal order.name "date" %}
     <a class="colactive"
+     href="{% listurl order=order.reversed_name %}"><img
+     src="/images/16-arrow-{% if order.reversed %}up{% else %}down{%endif%}.png"
+     width="16" height="16"
+    ></a> <a class="colactive"
      href="{% listurl order=order.reversed_name %}">Date</a>
    {% else %}
     <a class="colinactive" href="{% listurl order="date" %}">Date</a>
@@ -49,7 +57,11 @@
   <th>
    {% ifequal order.name "submitter" %}
     <a class="colactive"
-     href="{% listurl order=order.reversed_name %}">Submiter</a>
+     href="{% listurl order=order.reversed_name %}"><img
+     src="/images/16-arrow-{% if order.reversed %}up{% else %}down{%endif%}.png"
+     width="16" height="16"
+    ></a> <a class="colactive"
+     href="{% listurl order=order.reversed_name %}">Submitter</a>
    {% else %}
     <a class="colinactive" href="{% listurl order="submitter" %}">Submitter</a>
    {% endifequal %}
@@ -58,6 +70,10 @@
   <th>
    {% ifequal order.name "state" %}
     <a class="colactive"
+     href="{% listurl order=order.reversed_name %}"><img
+     src="/images/16-arrow-{% if order.reversed %}up{% else %}down{%endif%}.png"
+     width="16" height="16"
+    ></a> <a class="colactive"
      href="{% listurl order=order.reversed_name %}">State</a>
    {% else %}
     <a class="colinactive" href="{% listurl order="state" %}">State</a>