From e4f63d51b226b70342277a882b94fbc4770e93be Mon Sep 17 00:00:00 2001
From: Dan McGee <dan@archlinux.org>
Date: Sun, 31 Jan 2010 11:50:34 -0600
Subject: Fix up the packages search page

* Don't show checkboxes for non-authenticated users
* Fix up the adopt buttons to not be a table row
* Fix the form which didn't have a closing tag

Signed-off-by: Dan McGee <dan@archlinux.org>
---
 templates/packages/search.html | 25 ++++++++++++++-----------
 1 file changed, 14 insertions(+), 11 deletions(-)

(limited to 'templates')

diff --git a/templates/packages/search.html b/templates/packages/search.html
index 93b18b28..67d61fbb 100644
--- a/templates/packages/search.html
+++ b/templates/packages/search.html
@@ -39,10 +39,11 @@ <h4 style="text-align: right">Search Criteria</h4>
 
     {% if package_list %}
     <div class="greybox">
+        <form method="post" action="/packages/update/">
         <table class="results" width="100%">
             {% if paginator %}
                 <tr>
-                    <td colspan="6">
+                    <td colspan="{% if user.is_authenticated %}6{% else %}5{% endif %}">
                         {{paginator.count}} packages found.
                         Page {{page_obj.number}} of {{paginator.num_pages}}.
                     </td>
@@ -67,8 +68,9 @@ <h4 style="text-align: right">Search Criteria</h4>
                 </tr>
             {% endif %}
             <tr>
-                <form method="post" action="/packages/update/">
+                {% if user.is_authenticated %}
                 <th>&nbsp;</th>
+                {% endif %}
                 <th><a href="/packages/{% buildsortqs "arch" %}">Arch</a></th>
                 <th><a href="/packages/{% buildsortqs "repo" %}">Repo</a></th>
                 <th><a href="/packages/{% buildsortqs "pkgname" %}">Name</a></th>
@@ -80,7 +82,9 @@ <h4 style="text-align: right">Search Criteria</h4>
             </tr>
             {% for pkg in package_list %}
                 <tr class="{% cycle pkgr2,pkgr1 %}">
+                    {% if user.is_authenticated %}
                     <td><input type="checkbox" name="pkgid" value="{{ pkg.id }}" /></td>
+                    {% endif %}
                     <td>{{ pkg.arch.name }}</td>
                     <td>{{ pkg.repo.name|capfirst }}</td>
                     <td><a href="{{ pkg.get_absolute_url }}">{{ pkg.pkgname }}</a></td>
@@ -96,7 +100,7 @@ <h4 style="text-align: right">Search Criteria</h4>
             {% endfor %}
             {% if paginator %}
                 <tr>
-                    <td colspan="6">
+                    <td colspan="{% if user.is_authenticated %}6{% else %}5{% endif %}">
                         {{paginator.count}} packages found.
                         Page {{page_obj.number}} of {{paginator.num_pages}}.
                     </td>
@@ -120,15 +124,14 @@ <h4 style="text-align: right">Search Criteria</h4>
                     </td>
                 </tr>
             {% endif %}
-            {% if user.is_authenticated %}
-              <tr>
-                  <td colspan="3">&nbsp;</td>
-                  <td colspan="2" style="text-align:center"><input type="submit" name="adopt" value="Adopt Packages"></td>
-                  <td colspan="1" style="text-align:center"><input type="submit" name="disown" value="Disown Packages"></td>
-                  <td colspan="1">&nbsp;</td>
-              </tr>
-            {% endif %}
         </table>
+        {% if user.is_authenticated %}
+        <div id="adoptbox">
+            <input type="submit" name="adopt" value="Adopt Packages">
+            <input type="submit" name="disown" value="Disown Packages">
+        </div>
+        {% endif %}
+        </form>
     </div>
     {% endif %}
 {% endblock %}
-- 
cgit v1.2.3-54-g00ecf