From 8383a071608329c7683f7a710030ce945bd20b4d Mon Sep 17 00:00:00 2001
From: Dan McGee <dan@archlinux.org>
Date: Sun, 8 Jul 2012 23:30:48 -0500
Subject: Add a new jquery_tablesorter CDN template tag

And use it everywhere we were including the file before. This should
make updating the version a heck of a lot easier.

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

(limited to 'templates/packages/signoffs.html')

diff --git a/templates/packages/signoffs.html b/templates/packages/signoffs.html
index b20483c0..fc6adca2 100644
--- a/templates/packages/signoffs.html
+++ b/templates/packages/signoffs.html
@@ -81,8 +81,7 @@
         </tbody>
     </table>
 </div>
-{% load cdn %}{% jquery %}
-<script type="text/javascript" src="{% static "jquery.tablesorter.min.js" %}"></script>
+{% load cdn %}{% jquery %}{% jquery_tablesorter %}
 <script type="text/javascript" src="{% static "archweb.js" %}"></script>
 <script type="text/javascript">
 $(document).ready(function() {
-- 
cgit v1.2.3-2-g168b


From 8a8542ede6493939bd6528a72b8fd912fdf4d14b Mon Sep 17 00:00:00 2001
From: Dan McGee <dan@archlinux.org>
Date: Tue, 11 Dec 2012 10:02:26 -0600
Subject: Use multiple separate document.ready() handlers

If one of them breaks, we don't want to prevent the rest of the on-load
events from firing. This is currently a problem on some browsers with
the versions of jQuery and tablesorter we are using.

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

(limited to 'templates/packages/signoffs.html')

diff --git a/templates/packages/signoffs.html b/templates/packages/signoffs.html
index fc6adca2..b6737230 100644
--- a/templates/packages/signoffs.html
+++ b/templates/packages/signoffs.html
@@ -85,9 +85,11 @@
 <script type="text/javascript" src="{% static "archweb.js" %}"></script>
 <script type="text/javascript">
 $(document).ready(function() {
-    $('a.signoff-link').click(signoff_package);
-    $(".results").tablesorter({widgets: ['zebra'], sortList: [[0,0]],
+    $('.results').tablesorter({widgets: ['zebra'], sortList: [[0,0]],
         headers: { 5: { sorter: 'epochdate' }, 7: { sorter: false }, 8: {sorter: false } } });
+});
+$(document).ready(function() {
+    $('a.signoff-link').click(signoff_package);
     $('#signoffs_filter input').change(filter_signoffs);
     $('#criteria_reset').click(filter_signoffs_reset);
     // fire function on page load to ensure the current form selections take effect
-- 
cgit v1.2.3-2-g168b


From 040e0ddea591702856ce7d737bf4ea31f60a89e5 Mon Sep 17 00:00:00 2001
From: Dan McGee <dan@archlinux.org>
Date: Tue, 11 Dec 2012 20:53:50 -0600
Subject: Collapse really long signoff specifications using JS

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

(limited to 'templates/packages/signoffs.html')

diff --git a/templates/packages/signoffs.html b/templates/packages/signoffs.html
index b6737230..20371f95 100644
--- a/templates/packages/signoffs.html
+++ b/templates/packages/signoffs.html
@@ -69,7 +69,7 @@
                 {% endif %}
                 {% endif %}
                 <td>{% include "packages/signoff_cell.html" %}</td>
-                <td class="wrap">{% if not group.default_spec %}{% with group.specification as spec %}{% comment %}
+                <td class="wrap note">{% if not group.default_spec %}{% with group.specification as spec %}{% comment %}
                     {% endcomment %}{% if spec.required != 2 %}Required signoffs: {{ spec.required }}<br/>{% endif %}{% comment %}
                     {% endcomment %}{% if not spec.enabled %}Signoffs are not currently enabled<br/>{% endif %}{% comment %}
                     {% endcomment %}{% if spec.known_bad %}Package is known to be bad<br/>{% endif %}{% comment %}
@@ -95,5 +95,8 @@ $(document).ready(function() {
     // fire function on page load to ensure the current form selections take effect
     filter_signoffs();
 });
+$(document).ready(function() {
+    collapseNotes('.note');
+});
 </script>
 {% endblock %}
-- 
cgit v1.2.3-2-g168b


From b519c40e0af5b86ba660cf9d9cfc53a66fd8bef8 Mon Sep 17 00:00:00 2001
From: Dan McGee <dan@archlinux.org>
Date: Sat, 20 Apr 2013 11:11:04 -0500
Subject: Remove {% spaceless %} tag from a few more places

This tag is simply not worth the time it takes to do what it does.

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

(limited to 'templates/packages/signoffs.html')

diff --git a/templates/packages/signoffs.html b/templates/packages/signoffs.html
index 20371f95..2879c30b 100644
--- a/templates/packages/signoffs.html
+++ b/templates/packages/signoffs.html
@@ -50,9 +50,7 @@
             </tr>
         </thead>
         <tbody id="tbody_signoffs">
-            {% for group in signoff_groups %}
-            <tr class="{% cycle 'odd' 'even' %} {{ group.arch.name }} {{ group.target_repo|lower }}">
-                {% spaceless %}
+            {% for group in signoff_groups %}<tr class="{% cycle 'odd' 'even' %} {{ group.arch.name }} {{ group.target_repo|lower }}">
                 <td>{% pkg_details_link group.package %} {{ group.version }}</td>
                 <td>{{ group.arch.name }}</td>
                 <td>{{ group.target_repo }}</td>
@@ -75,9 +73,7 @@
                     {% endcomment %}{% if spec.known_bad %}Package is known to be bad<br/>{% endif %}{% comment %}
                     {% endcomment %}{{ spec.comments|default:""|linebreaksbr }}
                 {% endwith %}{% endif %}</td>
-                {% endspaceless %}
-            </tr>
-            {% endfor %}
+            </tr>{% endfor %}
         </tbody>
     </table>
 </div>
-- 
cgit v1.2.3-2-g168b


From 3e085183bf39c59fdcd0b9e67b5a02653a51d30e Mon Sep 17 00:00:00 2001
From: Dan McGee <dan@archlinux.org>
Date: Sat, 20 Apr 2013 11:28:12 -0500
Subject: Move all script blocks into {% script_block %} section

I added this a while back, but didn't roll it out to all templates.

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

(limited to 'templates/packages/signoffs.html')

diff --git a/templates/packages/signoffs.html b/templates/packages/signoffs.html
index 2879c30b..6fbbd00e 100644
--- a/templates/packages/signoffs.html
+++ b/templates/packages/signoffs.html
@@ -77,6 +77,9 @@
         </tbody>
     </table>
 </div>
+{% endblock %}
+
+{% block script_block %}
 {% load cdn %}{% jquery %}{% jquery_tablesorter %}
 <script type="text/javascript" src="{% static "archweb.js" %}"></script>
 <script type="text/javascript">
-- 
cgit v1.2.3-2-g168b