From 663aa383ffb28ba9e6d5ee2f770b47ba3ed97e99 Mon Sep 17 00:00:00 2001
From: Thayer Williams <thayerw@gmail.com>
Date: Tue, 16 Mar 2010 11:58:29 -0700
Subject: Redesigned Signoffs page

* descriptive href titles
* better semantics (page titles, etc)
* added tbody for prettier printing
* proper django cycling

Signed-off-by: Dan McGee <dan@archlinux.org>
---
 templates/packages/signoffs.html | 43 ++++++++++++++++++++++++----------------
 1 file changed, 26 insertions(+), 17 deletions(-)

diff --git a/templates/packages/signoffs.html b/templates/packages/signoffs.html
index cb6a848e..930b606e 100644
--- a/templates/packages/signoffs.html
+++ b/templates/packages/signoffs.html
@@ -2,9 +2,13 @@
 {% block title %}Arch Linux - Package Signoffs{% endblock %}
 
 {% block content %}
-    {% if packages %}
-    <div class="greybox">
-        <table class="results" width="100%">
+{% if packages %}
+<div id="dev-signoffs" class="box">
+
+    <h2>Package Signoffs</h2>
+
+    <table id="signoffs" class="results">
+        <thead>
             <tr>
                 <th>Arch</th>
                 <th>Package</th>
@@ -14,28 +18,33 @@
                 <th>Approved</th>
                 <th>Signoff</th>
             </tr>
-        {% for pkg,target in packages %}
-        <tr class="{% cycle pkgr2,pkgr1 %}">
+        </thead>
+        <tbody>
+            {% for pkg,target in packages %}
+            <tr class="{% cycle 'odd' 'even' %}">
                 <td>{{ pkg.arch.name }}</td>
-                <td><a href="{{ pkg.get_absolute_url }}">{{ pkg.pkgname }}</a></td>
+                <td><a href="{{ pkg.get_absolute_url }}"
+                        title="View package details for {{ pkg.pkgname }}">{{ pkg.pkgname }}</a></td>
                 <td>{{ pkg.pkgver }}-{{ pkg.pkgrel }}</td>
                 <td>{{ pkg.last_update }}</td>
                 <td>{{ target }}</td>
-                <td class="signoff_{{pkg.approved_for_signoff|yesno}}">{{ pkg.approved_for_signoff|yesno:"Yes,No" }}</td>
+                <td class="signoff-{{pkg.approved_for_signoff|yesno}}">
+                    {{ pkg.approved_for_signoff|yesno:"Yes,No" }}</td>
                 <td>
                     <ul>
-                    {% for signoff in pkg.signoffs %}
-                    <li>{{signoff.packager}}</li>
-                    {% endfor %}
-                    <li>
-                    <a href="/packages/signoff_package/{{pkg.arch}}/{{pkg.pkgname}}/">Signoff</a>
-                    </li>
+                        <li><a href="/packages/signoff_package/{{pkg.arch}}/{{pkg.pkgname}}/"
+                            title="Signoff {{pkg.pkgname}} for {{pkg.arch}}">Signoff</a>
+                        </li>
+                        {% for signoff in pkg.signoffs %}
+                        <li class="signed-username" title="Signed off by {{signoff.packager}}">
+                        {{signoff.packager}}</li>
+                        {% endfor %}
                     </ul>
                 </td>
             </tr>
         {% endfor %}
-        </table>
-    </div>
-    {% endif %}
+        </tbody>
+    </table>
+</div>
+{% endif %}
 {% endblock %}
-
-- 
cgit v1.2.3-55-g3dc8