html: Drop type attribute value of text/javascript

As is encouraged[1]

https://developer.mozilla.org/en-US/docs/Web/HTML/Reference/Elements/script#attribute_is_not_set_default_an_empty_string_or_a_javascript_mime_type

Links:

- Apps page loads and javascript works as expected.

Reviewed-by: James Valleroy <jvalleroy@mailbox.org>

plinth/modules/backups/templates/backups_add_remote_repository.html

@@ -8,7 +8,7 @@
 {% load static %}
 
 {% block page_js %}
-  <script type="text/javascript" src="{% static 'backups/backups_add_remote_repository.js' %}"
+  <script src="{% static 'backups/backups_add_remote_repository.js' %}"
           defer></script>
 {% endblock %}
 

plinth/modules/dynamicdns/templates/dynamicdns-domain.html

@@ -6,6 +6,5 @@
 {% load static %}
 
 {% block page_js %}
-  <script type="text/javascript" src="{% static 'dynamicdns/dynamicdns.js' %}"
-          defer></script>
+  <script src="{% static 'dynamicdns/dynamicdns.js' %}" defer></script>
 {% endblock %}

plinth/modules/ejabberd/templates/ejabberd.html

@@ -8,8 +8,7 @@
 {% load static %}
 
 {% block page_js %}
-  <script type="text/javascript" src="{% static 'ejabberd/ejabberd.js' %}"
-          defer></script>
+  <script src="{% static 'ejabberd/ejabberd.js' %}" defer></script>
 {% endblock %}
 
 {% block status %}

plinth/modules/janus/templates/janus_video_room.html

@@ -29,26 +29,16 @@
     <meta name="viewport" content="width=device-width, initial-scale=1">
     <title>Janus Video Room</title>
 
-    <script type="text/javascript"
-            src="/javascript/webrtc-adapter/adapter.min.js" ></script>
-    <script type="text/javascript"
-            src="/javascript/jquery/jquery.min.js" ></script>
-    <script type="text/javascript"
-            src="/javascript/popperjs2/popper.min.js"></script>
-    <script type="text/javascript"
-            src="/javascript/jquery-blockui/jquery.blockUI.min.js" ></script>
-    <script type="text/javascript"
-            src="/javascript/bootstrap5/js/bootstrap.min.js" ></script>
-    <script type="text/javascript"
-            src="/javascript/bootbox/bootbox.min.js" ></script>
-    <script type="text/javascript"
-            src="/javascript/toastr/toastr.min.js" ></script>
-    <script type="text/javascript"
-            src="{% static 'janus/janus-freedombox-config.js' %}" ></script>
-    <script type="text/javascript"
-            src="/javascript/janus-gateway/janus.min.js" ></script>
-    <script type="text/javascript"
-            src="{% static 'janus/janus-video-room.js' %}" ></script>
+    <script src="/javascript/webrtc-adapter/adapter.min.js" ></script>
+    <script src="/javascript/jquery/jquery.min.js" ></script>
+    <script src="/javascript/popperjs2/popper.min.js"></script>
+    <script src="/javascript/jquery-blockui/jquery.blockUI.min.js" ></script>
+    <script src="/javascript/bootstrap5/js/bootstrap.min.js" ></script>
+    <script src="/javascript/bootbox/bootbox.min.js" ></script>
+    <script src="/javascript/toastr/toastr.min.js" ></script>
+    <script src="{% static 'janus/janus-freedombox-config.js' %}" ></script>
+    <script src="/javascript/janus-gateway/janus.min.js" ></script>
+    <script src="{% static 'janus/janus-video-room.js' %}" ></script>
 
     <link rel="stylesheet"
           href="/javascript/bootstrap5/css/bootstrap.css"

plinth/modules/matrixsynapse/templates/matrix-synapse.html

@@ -7,8 +7,7 @@
 {% load static %}
 
 {% block page_js %}
-  <script type="text/javascript" src="{% static 'matrixsynapse/matrixsynapse.js' %}"
-          defer></script>
+  <script src="{% static 'matrixsynapse/matrixsynapse.js' %}" defer></script>
 {% endblock %}
 
 {% block status %}

plinth/modules/mediawiki/templates/mediawiki.html

@@ -7,6 +7,5 @@
 {% load static %}
 
 {% block page_js %}
-  <script type="text/javascript" src="{% static 'mediawiki/mediawiki.js' %}"
-          defer></script>
+  <script src="{% static 'mediawiki/mediawiki.js' %}" defer></script>
 {% endblock %}

plinth/modules/networks/templates/connections_edit.html

@@ -7,8 +7,7 @@
 {% load static %}
 
 {% block page_js %}
-  <script type="text/javascript" src="{% static 'networks/networks.js' %}"
-          defer></script>
+  <script src="{% static 'networks/networks.js' %}" defer></script>
 {% endblock %}
 
 {% block content %}

plinth/modules/sharing/templates/sharing_add_edit.html

@@ -8,8 +8,7 @@
 {% load static %}
 
 {% block page_js %}
-  <script type="text/javascript" src="{% static 'sharing/sharing_add_edit.js' %}"
-          defer></script>
+  <script src="{% static 'sharing/sharing_add_edit.js' %}" defer></script>
 {% endblock %}
 
 {% block content %}

plinth/modules/snapshot/templates/snapshot_manage.html

@@ -8,8 +8,7 @@
 {% load static %}
 
 {% block page_js %}
-  <script type="text/javascript" src="{% static 'snapshot/snapshot.js' %}"
-          defer></script>
+  <script src="{% static 'snapshot/snapshot.js' %}" defer></script>
 {% endblock %}
 
 {% block configuration %}

plinth/modules/tor/templates/tor.html

@@ -8,8 +8,7 @@
 {% load static %}
 
 {% block page_js %}
-  <script type="text/javascript" src="{% static 'tor/tor.js' %}"
-          defer></script>
+  <script src="{% static 'tor/tor.js' %}" defer></script>
 {% endblock %}
 
 {% block status %}

plinth/modules/torproxy/templates/torproxy.html

@@ -6,6 +6,5 @@
 {% load static %}
 
 {% block page_js %}
-  <script type="text/javascript" src="{% static 'torproxy/torproxy.js' %}"
-          defer></script>
+  <script src="{% static 'torproxy/torproxy.js' %}" defer></script>
 {% endblock %}

plinth/modules/users/templates/users_update.html

@@ -8,8 +8,7 @@
 {% load static %}
 
 {% block page_js %}
-  <script type="text/javascript" src="{% static 'users/users.js' %}"
-          defer></script>
+  <script src="{% static 'users/users.js' %}" defer></script>
 {% endblock %}
 
 {% block content %}

plinth/templates/apps.html

@@ -7,7 +7,7 @@
 {% load i18n %}
 
 {% block page_js %}
-  <script type="text/javascript" src="{% static 'tags.js' %}" defer></script>
+  <script src="{% static 'tags.js' %}" defer></script>
 {% endblock %}
 
 {% block body_class %}apps-page{% endblock %}

plinth/templates/base.html

@@ -60,13 +60,13 @@
 
   <!-- This script is not loaded in defer mode because it needs to run before
        page is rendered -->
-  <script type="text/javascript" src="{% static 'theme/js/color-modes.js' %}"></script>
+  <script src="{% static 'theme/js/color-modes.js' %}"></script>
   <!-- Local link to system Bootstrap JS -->
-  <script type="text/javascript" src="{% static '/javascript/popperjs2/popper.min.js' %}" defer></script>
-  <script type="text/javascript" src="{% static '/javascript/bootstrap5/js/bootstrap.bundle.min.js' %}" defer></script>
-  <script type="text/javascript" src="{% static '/javascript/htmx/htmx.min.js' %}" defer></script>
+  <script src="{% static '/javascript/popperjs2/popper.min.js' %}" defer></script>
+  <script src="{% static '/javascript/bootstrap5/js/bootstrap.bundle.min.js' %}" defer></script>
+  <script src="{% static '/javascript/htmx/htmx.min.js' %}" defer></script>
 
-  <script type="text/javascript" src="{% static 'theme/js/main.js' %}" defer></script>
+  <script src="{% static 'theme/js/main.js' %}" defer></script>
   {% block app_js %}<!-- placeholder for app-specific js files -->{% endblock %}
   {% block page_js %}<!-- placeholder for page-specific js files -->{% endblock %}
 

plinth/templates/system.html

@@ -7,7 +7,7 @@
 {% load i18n %}
 
 {% block page_js %}
-  <script type="text/javascript" src="{% static 'tags.js' %}" defer></script>
+  <script src="{% static 'tags.js' %}" defer></script>
 {% endblock %}
 
 {% block body_class %}system-page{% endblock %}