Use up/down arrows as sorting indicator

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

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

diff --git a/htdocs/css/style.css b/htdocs/css/style.css
index 3b10edb080f515d2085bee167382c2079bc2c5f1..3604148e0519e565f1efef4ab8ca30a67d5260bd 100644 (file)
--- a/htdocs/css/style.css
+++ b/htdocs/css/style.css
@@ -124,6 +124,10 @@ table.patchlist th {
        padding-left: 6px;
 }
 
        padding-left: 6px;
 }
 

+table.patchlist th img {
+       vertical-align: bottom;
+}
+

 table.patchlist td {
        padding: 2px 6px 2px 6px;
        margin: 0px;
 table.patchlist td {
        padding: 2px 6px 2px 6px;
        margin: 0px;


@@ -145,14 +149,11 @@ table.patchlist tr.even {
        background: #e8e8e8;
 }
 
        background: #e8e8e8;
 }
 

-a.colactive {
-       color: red;
-}
-
-a.colinactive {
+a.colinactive, a.colactive {

        color: black;
        text-decoration: none;
 }
        color: black;
        text-decoration: none;
 }

+

 a.colinactive:hover {
        color: red;
 }
 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 (file)
index 0000000..fa0ff7b
Binary files /dev/null and b/htdocs/images/16-arrow-down.png differ

diff --git a/htdocs/images/16-arrow-up.png b/htdocs/images/16-arrow-up.png
new file mode 100644 (file)
index 0000000..60c4eb9
Binary files /dev/null and b/htdocs/images/16-arrow-up.png differ

diff --git a/templates/patchwork/help/about.html b/templates/patchwork/help/about.html
index fa5094ffb5d110ce47af3f59c38992f60f38fc98..4cae691225c1f404c4d9eeaae2367bf5c6e0bef4 100644 (file)
--- 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>
 
 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 %}
 
 {% endblock %}
 

diff --git a/templates/patchwork/patch-list.html b/templates/patchwork/patch-list.html
index a0a8cec98a7c579bbcb729302626f33fa6774352..57491e20ccc7fdd15019425b204ff4e6beb5e57c 100644 (file)
--- a/templates/patchwork/patch-list.html
+++ b/templates/patchwork/patch-list.html
@@ -31,6 +31,10 @@
   <th>
    {% ifequal order.name "name" %}
     <a class="colactive"
   <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>
      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"
   <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>
      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"
   <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 %}
    {% else %}
     <a class="colinactive" href="{% listurl order="submitter" %}">Submitter</a>
    {% endifequal %}


@@ -58,6 +70,10 @@
   <th>
    {% ifequal order.name "state" %}
     <a class="colactive"
   <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>
      href="{% listurl order=order.reversed_name %}">State</a>
    {% else %}
     <a class="colinactive" href="{% listurl order="state" %}">State</a>