From 04e51503f690626118e8071cd5f8f7afd9ad4cf1 Mon Sep 17 00:00:00 2001
From: Jeremy Kerr <jk@ozlabs.org>
Date: Sat, 23 Aug 2008 11:31:10 +0800
Subject: [PATCH] Use up/down arrows as sorting indicator

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

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
---
 htdocs/css/style.css                |  11 ++++++-----
 htdocs/images/16-arrow-down.png     | Bin 0 -> 354 bytes
 htdocs/images/16-arrow-up.png       | Bin 0 -> 332 bytes
 templates/patchwork/help/about.html |   2 ++
 templates/patchwork/patch-list.html |  18 +++++++++++++++++-
 5 files changed, 25 insertions(+), 6 deletions(-)
 create mode 100644 htdocs/images/16-arrow-down.png
 create mode 100644 htdocs/images/16-arrow-up.png

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 0000000000000000000000000000000000000000..fa0ff7b3204cde4110c41b9c907896325697401f
GIT binary patch
literal 354
zcmV-o0iFJdP)<h;3K|Lk000e1NJLTq000mG000mO1^@s6AM^iV0000PbVXQnQ*UN;
zcVTj606}DLVr3vnZDD6+Qe|Oed2z{QJOBUz1W80eRCwBA{Qv(y10{fQa2c?3$2x{P
z-}m7)QH9fq(cL2m&SycE1Zmb*v}Ul?3u1s63^R+7fsv7s;qaw(3|H^(`|r*W#E8`Z
z5cv1+FT;;tpBeuB|HA-t9Y`}1BMVp#9WbFeukrfB6AVB9d}8?a>m9?_pRd6f#0H6h
z<Une08UO?hAmY^ZT@1hfd}jFh`#l(g*dQ@vH8{Nhvf<8?ef84v-x<FDegzieWDsMx
z`{XW$?buuZx8dIX&kStL{9p`XW3wHX0brY+JbT5!%_alJAT~DJaT)-#>F(XPV2sOl
zd<KARy!+%f0|?`?9cv&Wy8@^YukC{)GXVq`0J1BIBu0_2dH?_b07*qoM6N<$g4+9<
A0RR91

literal 0
HcmV?d00001

diff --git a/htdocs/images/16-arrow-up.png b/htdocs/images/16-arrow-up.png
new file mode 100644
index 0000000000000000000000000000000000000000..60c4eb9cff183104e4a0c7dfa2699bc1d6dfd88a
GIT binary patch
literal 332
zcmV-S0ki&zP)<h;3K|Lk000e1NJLTq000mG000mO1^@s6AM^iV0000PbVXQnQ*UN;
zcVTj606}DLVr3vnZDD6+Qe|Oed2z{QJOBUy?ny*JRCwBA{Qv(y10{fQa2c?3$2t_r
zJKy(#B~>_`7?H)@J%ZqDCR`?iG;1qbGk`G6AZ&mIpYzN$J;3zU`$uuAVZvp*nyewi
z*I#eI7{tcq3Y=chQ8r@u_VX1OgV;oS0c4Ymlo-RaADb9{{(TP?V`mm*0Eyjsw(mcB
zfMRulvZ4UPpMT#N{{8y}#vnFGjGz}lwkycUGcYnTF)%Z6Ft9T7f-#5<5(CMhhXo6!
ziEIoUETRmIK&gNKe?i0;*%;Va1c7od@P`4=e~ovZ?5n@`<TlPg1F6C1f(D=hLaDc5
eaAhWd00RI#MQARJ**GZx0000<MNUMnLSTZi$%d=|

literal 0
HcmV?d00001

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>
-- 
2.39.2