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/differences.html | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

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

diff --git a/templates/packages/differences.html b/templates/packages/differences.html
index 01e1f857..8d6887a3 100644
--- a/templates/packages/differences.html
+++ b/templates/packages/differences.html
@@ -95,8 +95,7 @@
 
 </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 b1206a4109d3b13b06c3116ef3a2dd6c9e125a1f Mon Sep 17 00:00:00 2001
From: Dan McGee <dan@archlinux.org>
Date: Mon, 23 Jul 2012 22:22:49 -0500
Subject: Enable filtering of todolist packages

This matches the filtering options we have on the signoffs and package
differences pages.

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

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

diff --git a/templates/packages/differences.html b/templates/packages/differences.html
index 8d6887a3..b0c5b539 100644
--- a/templates/packages/differences.html
+++ b/templates/packages/differences.html
@@ -8,7 +8,7 @@
 {% if differences %}
 <div class="box">
     <h2>Package Differences by Architecture</h2>
-    <div id="differences-filter" class="filter-criteria">
+    <div class="filter-criteria">
         <h3>Filter Differences View</h3>
         <form id="diff_filter" method="post" action=".">
         <fieldset>
@@ -29,7 +29,6 @@
         </fieldset>
         </form>
     </div>
-    {# TODO some sort of spacing here #}
 
     <table id="table_differences" class="results">
         <thead>
-- 
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/differences.html | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

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

diff --git a/templates/packages/differences.html b/templates/packages/differences.html
index b0c5b539..61906fef 100644
--- a/templates/packages/differences.html
+++ b/templates/packages/differences.html
@@ -99,13 +99,14 @@
 <script type="text/javascript">
 $(document).ready(function() {
     $('#table_differences').tablesorter({widgets: ['zebra'], sortList: [[1,0], [0,0]]});
+    $('#table_multilib_differences').tablesorter({widgets: ['zebra'], sortList: [[5, 0]]});
+});
+$(document).ready(function() {
     $('#diff_filter select').change(filter_packages);
     $('#diff_filter input').change(filter_packages);
     $('#criteria_reset').click(filter_packages_reset);
     // fire function on page load to ensure the current form selections take effect
     filter_packages();
-
-    $('#table_multilib_differences').tablesorter({widgets: ['zebra'], sortList: [[5, 0]]});
 });
 </script>
 {% endif %}
-- 
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/differences.html | 2 ++
 1 file changed, 2 insertions(+)

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

diff --git a/templates/packages/differences.html b/templates/packages/differences.html
index 61906fef..3d3f6edf 100644
--- a/templates/packages/differences.html
+++ b/templates/packages/differences.html
@@ -93,7 +93,9 @@
     </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


From 7873cb9a052c7991f3d91ddf4dc05caef6305cd7 Mon Sep 17 00:00:00 2001
From: Dan McGee <dan@archlinux.org>
Date: Sat, 20 Apr 2013 13:31:20 -0500
Subject: Unbreak the package differences page

We had a weird conditional around everything on the page that doesn't
really need to be there. Remove it.

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

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

diff --git a/templates/packages/differences.html b/templates/packages/differences.html
index 3d3f6edf..b0b9b419 100644
--- a/templates/packages/differences.html
+++ b/templates/packages/differences.html
@@ -5,7 +5,6 @@
 {% block navbarclass %}anb-packages{% endblock %}
 
 {% block content %}
-{% if differences %}
 <div class="box">
     <h2>Package Differences by Architecture</h2>
     <div class="filter-criteria">
@@ -111,5 +110,4 @@ $(document).ready(function() {
     filter_packages();
 });
 </script>
-{% endif %}
 {% endblock %}
-- 
cgit v1.2.3-2-g168b