views: Move js code to separate file

Move patch-list related JS code to a new patch-list.js file, to
make the JavaScript easy to read and change in one place. This makes
automatic code formatting easier, makes it more straightforward to
measure test coverage and discover opportunities for refactoring, and
simplifies a possible future migration to TypeScript if the project
chooses to go in that direction.

No user-visible change should be noticed.

Signed-off-by: Raxel Gutierrez <raxel@google.com>
Signed-off-by: Stephen Finucane <stephen@that.guru>
[stephenfin: Addressed merged conflicts]

diff --git a/htdocs/README.rst b/htdocs/README.rst
index 22fe58d1f38869cf69de42b9f3c1d73f409b0c4b..9a5211233a62560d46066779235df88fa613fc43 100644 (file)
--- a/htdocs/README.rst
+++ b/htdocs/README.rst
@@ -122,6 +122,12 @@ js
   :GitHub: https://github.com/js-cookie/js-cookie/
   :Version: 3.0.0
 
+``patch-list.js.``
+  Event helpers and other application logic for patch-list.html. These
+  support patch list manipulation.
+
+  Part of Patchwork.
+
 ``rest.js.``
   Utility module for REST API requests to be used by other Patchwork JS files.
 

diff --git a/htdocs/js/patch-list.js b/htdocs/js/patch-list.js
new file mode 100644 (file)
index 0000000..6be031a
--- /dev/null
+++ b/htdocs/js/patch-list.js
@@ -0,0 +1,14 @@
+$( document ).ready(function() {
+    $("#patch-list").stickyTableHeaders();
+
+    $("#patch-list").checkboxes("range", true);
+
+    $("#check-all").change(function(e) {
+        if(this.checked) {
+            $("#patch-list > tbody").checkboxes("check");
+        } else {
+            $("#patch-list > tbody").checkboxes("uncheck");
+        }
+        e.preventDefault();
+    });
+});

diff --git a/patchwork/templates/patchwork/partials/patch-list.html b/patchwork/templates/patchwork/partials/patch-list.html
index 48b81f3d5cbd460c718318c07f2a431363cc1153..1bdef36d23b03747a374befc91467efdccd27f87 100644 (file)
--- a/patchwork/templates/patchwork/partials/patch-list.html
+++ b/patchwork/templates/patchwork/partials/patch-list.html
@@ -4,6 +4,10 @@
 {% load project %}
 {% load static %}
 
+{% block headers %}
+  <script src="{% static "js/patch-list.js" %}"></script>
+{% endblock %}
+
 {% include "patchwork/partials/filters.html" %}
 {% include "patchwork/partials/pagination.html" %}
 
@@ -32,23 +36,6 @@
 </div>
 {% endif %}
 
-<script type="text/javascript">
-$(document).ready(function() {
-    $('#patch-list').stickyTableHeaders();
-
-    $('#patch-list').checkboxes('range', true);
-
-    $('#check-all').change(function(e) {
-        if(this.checked) {
-            $('#patch-list > tbody').checkboxes('check');
-        } else {
-            $('#patch-list > tbody').checkboxes('uncheck');
-        }
-        e.preventDefault();
-    });
-});
-</script>
-
 <form method="post">
   {% csrf_token %}
   <input type="hidden" name="form" value="patch-list-form"/>