templates: Get rid of type attribute in script tags

In HTML5, the type attribute of a script tag is optional if it's
JavaScript.

Remove all occurrences. The only real gain is slightly smaller page output,
but it also shuts up validators that like to be noisy about this.

Signed-off-by: Andrew Donnellan <ajd@linux.ibm.com>
Signed-off-by: Daniel Axtens <dja@axtens.net>

diff --git a/patchwork/templates/patchwork/bundle.html b/patchwork/templates/patchwork/bundle.html
index b5c9f90a4afdb24e66f4ad9952e6fdc444bc3bf3..411c18b5b3450fe9116d194c7d05b9f421737855 100644 (file)
--- a/patchwork/templates/patchwork/bundle.html
+++ b/patchwork/templates/patchwork/bundle.html
@@ -4,8 +4,8 @@
 {% load static %}
 
 {% block headers %}
-  <script type="text/javascript" src="{% static "js/jquery.tablednd.js" %}"></script>
-  <script type="text/javascript" src="{% static "js/bundle.js" %}"></script>
+  <script src="{% static "js/jquery.tablednd.js" %}"></script>
+  <script src="{% static "js/bundle.js" %}"></script>
 {% endblock %}
 {% block title %}{{project.name}}{% endblock %}
 

diff --git a/patchwork/templates/patchwork/login.html b/patchwork/templates/patchwork/login.html
index e8b1a4e8f242d2cb5e957f956788005a7322cbc7..46f77edf698459f25addf03cad7df7593da5f298 100644 (file)
--- a/patchwork/templates/patchwork/login.html
+++ b/patchwork/templates/patchwork/login.html
@@ -4,7 +4,7 @@
 {% block heading %}Sign in to Patchwork{% endblock %}
 
 {% block headers %}
-  <script type="text/javascript">
+  <script>
     $(function() {
         $('#id_username').focus()
     });

diff --git a/patchwork/templates/patchwork/submission.html b/patchwork/templates/patchwork/submission.html
index b3849789b1fde99e286f9663c673d8c20043380a..2706a5535605bbf43067e668904487b435f0efed 100644 (file)
--- a/patchwork/templates/patchwork/submission.html
+++ b/patchwork/templates/patchwork/submission.html
@@ -8,7 +8,7 @@
 {% block title %}{{submission.name}}{% endblock %}
 
 {% block body %}
-<script type="text/javascript">
+<script>
 function toggle_div(link_id, headers_id)
 {
     var link = document.getElementById(link_id)

diff --git a/templates/base.html b/templates/base.html
index 802ea98b0745893b43edaff4f10e85849a69b673..01b0d6b5e598944abf9cee2fb876f280bb8b008a 100644 (file)
--- a/templates/base.html
+++ b/templates/base.html
@@ -7,9 +7,9 @@
   <link rel="stylesheet" type="text/css" href="{% static "css/bootstrap.min.css" %}"/>
   <link rel="stylesheet" type="text/css" href="{% static "css/selectize.bootstrap3.css" %}"/>
   <link rel="stylesheet" type="text/css" href="{% static "css/style.css" %}"/>
-  <script type="text/javascript" src="{% static "js/jquery-1.10.1.min.js" %}"></script>
-  <script type="text/javascript" src="{% static "js/jquery.stickytableheaders.min.js" %}"></script>
-  <script type="text/javascript" src="{% static "js/jquery.checkboxes-1.0.6.min.js" %}"></script>
+  <script src="{% static "js/jquery-1.10.1.min.js" %}"></script>
+  <script src="{% static "js/jquery.stickytableheaders.min.js" %}"></script>
+  <script src="{% static "js/jquery.checkboxes-1.0.6.min.js" %}"></script>
   <!-- IE8 support of HTML5 elements and media queries -->
   <!-- WARNING: Respond.js doesn't work if you view the page via file:// -->
   <!--[if lt IE 9]>
@@ -18,10 +18,10 @@
     <script src="https://oss.maxcdn.com/respond/1.4.2/respond.min.js"></script>
     <script src="http://cdnjs.cloudflare.com/ajax/libs/es5-shim/2.0.8/es5-shim.min.js"></script>
   <![endif]-->
-  <script type="text/javascript" src="{% static "js/bootstrap.min.js" %}"></script>
-  <script type="text/javascript" src="{% static "js/selectize.min.js" %}"></script>
-  <script type="text/javascript" src="{% static "js/clipboard.min.js" %}"></script>
-  <script type="text/javascript">
+  <script src="{% static "js/bootstrap.min.js" %}"></script>
+  <script src="{% static "js/selectize.min.js" %}"></script>
+  <script src="{% static "js/clipboard.min.js" %}"></script>
+  <script>
    $(document).ready(function() {
        new Clipboard(document.querySelectorAll('button.btn-copy'));
    });