Merge branch 'maintenance/clean-up-js-files' into test

hypha/apply/determinations/templates/determinations/base_determination_form.html

@@ -86,7 +86,6 @@ <h2>{{ value }}</h2>
 {% endblock %}
 
 {% block extra_js %}
-    {{ form.media.js }}
     {{ field_blocks_ids|json_script:"block-ids" }}
     <script src="{% static 'js/determination-template.js' %}"></script>
 {% endblock %}

hypha/apply/funds/templates/funds/application_base.html

@@ -60,7 +60,7 @@ <h3>{% blocktrans %}Sorry this {{ page|verbose_name }} is not accepting applicat
                         {% endif %}
                     {% endfor %}
 
-                {# Hidden fields needed e.g. for django-file-form. See `StreamBaseForm.hidden_fields` #}
+                    {# Hidden fields needed e.g. for django-file-form. See `StreamBaseForm.hidden_fields` #}
                     {% for hidden_field in form.hidden_fields %}
                         {{ hidden_field }}
                     {% endfor %}
@@ -85,7 +85,6 @@ <h3>{% blocktrans %}Sorry this {{ page|verbose_name }} is not accepting applicat
 {% endblock %}
 
 {% block extra_js %}
-    {{ form.media.js }}
     <script src="{% static 'js/application-form.js' %}"></script>
     <script src="{% static 'js/tinymce-word-count.js' %}"></script>
     <script src="{% static 'js/submission-form-copy.js' %}"></script>

hypha/apply/funds/templates/funds/applicationsubmission_form.html

@@ -26,7 +26,7 @@
                     {% endif %}
                 {% endfor %}
 
-            {# Hidden fields needed e.g. for django-file-form. See `StreamBaseForm.hidden_fields` #}
+                {# Hidden fields needed e.g. for django-file-form. See `StreamBaseForm.hidden_fields` #}
                 {% for hidden_field in form.hidden_fields %}
                     {{ hidden_field }}
                 {% endfor %}
@@ -44,7 +44,6 @@
 {% endblock %}
 
 {% block extra_js %}
-    {{ form.media.js }}
     <script src="{% static 'js/application-form.js' %}"></script>
     <script src="{% static 'js/tinymce-word-count.js' %}"></script>
     <script src="{% static 'js/multi-input-fields.js' %}"></script>

hypha/apply/funds/templates/funds/includes/delegated_form_base.html

@@ -7,7 +7,6 @@
     {% if action %}action="{{ action }}"{% endif %}
 >
     {% csrf_token %}
-    {{ form.media.js }}
     {% for hidden in form.hidden_fields %}
         {{ hidden }}
     {% endfor %}

hypha/apply/funds/templates/funds/includes/dialog_form_base.html

@@ -10,7 +10,6 @@
     enctype="multipart/form-data"
 >
     {% csrf_token %}
-    {{ form.media.js }}
     {% for hidden in form.hidden_fields %}
         {{ hidden }}
     {% endfor %}

hypha/apply/funds/templates/funds/includes/translate_application_form.html

@@ -8,7 +8,6 @@
     hx-post="{{ request.path }}"
 >
     {% csrf_token %}
-    {{ form.media.js }}
     {% for hidden in form.hidden_fields %}
         {{ hidden }}
     {% endfor %}

hypha/apply/funds/templates/funds/includes/update_reviewer_form.html

@@ -9,7 +9,6 @@
     hx-post="{{ request.path }}"
 >
     {% csrf_token %}
-    {{ form.media.js }}
     {% for hidden in form.hidden_fields %}
         {{ hidden }}
     {% endfor %}

hypha/apply/projects/templates/application_projects/invoice_form.html

@@ -47,7 +47,3 @@
         </div>
     </div>
 {% endblock %}
-
-{% block extra_js %}
-    {{ form.media.js }}
-{% endblock %}

hypha/apply/projects/templates/application_projects/modals/report_frequency_config.html

@@ -42,7 +42,6 @@
         enctype="multipart/form-data"
     >
         {% csrf_token %}
-        {{ form.media.js }}
         {% for hidden in form.hidden_fields %}
             {{ hidden }}
         {% endfor %}

hypha/apply/projects/templates/application_projects/project_approval_form.html

@@ -31,7 +31,7 @@
                         {% endif %}
                     {% endfor %}
 
-                {# Hidden fields needed e.g. for django-file-form. See `StreamBaseForm.hidden_fields` #}
+                    {# Hidden fields needed e.g. for django-file-form. See `StreamBaseForm.hidden_fields` #}
                     {% for hidden_field in pf_form.hidden_fields %}
                         {{ hidden_field }}
                     {% endfor %}

hypha/apply/projects/templates/application_projects/project_form.html

@@ -36,7 +36,3 @@
         </div>
     </div>
 {% endblock %}
-
-{% block extra_js %}
-    {{ form.media.js }}
-{% endblock %}

hypha/apply/projects/templates/application_projects/report_form.html

@@ -76,6 +76,5 @@
 {% endblock %}
 
 {% block extra_js %}
-    {{ form.media.js }}
     <script src="{% static 'js/multi-input-fields.js' %}"></script>
 {% endblock %}

hypha/apply/review/templates/review/review_edit_form.html

@@ -80,6 +80,5 @@ <h2>{{ value }}</h2>
 {% endblock %}
 
 {% block extra_js %}
-    {{ form.media.js }}
     <script src="{% static 'js/review-form-actions.js' %}"></script>
 {% endblock %}

hypha/apply/review/templates/review/review_form.html

@@ -101,6 +101,5 @@ <h2>{{ value }}</h2>
 {% endblock %}
 
 {% block extra_js %}
-    {{ form.media.js }}
     <script src="{% static 'js/review-form-actions.js' %}"></script>
 {% endblock %}

hypha/apply/stream_forms/fields.py

@@ -20,7 +20,7 @@ def media(self):
                 ]
             },
             js=[
-                static("file_form/file_form.js"),
+                static("file_form/file_form.min.js"),
                 static("js/file-uploads.js"),
             ],
         )

hypha/static_src/javascript/file-uploads.js

@@ -1,26 +1,30 @@
-jQuery(function ($) {
+// We use htmx.onLoad() so it will initilise file uploads in htmx dialogs.
+htmx.onLoad(function () {
   // Initialize django-file-form
-  $("form")
-    .get()
-    .forEach(function (form) {
-      // Prevent initialising it multiple times and run it for forms
-      // that have a `form_id` field added by django-file-form.
-      if (!form.initUploadFieldsDone && form.querySelector("[name=form_id]")) {
-        init(form);
-        form.initUploadFieldsDone = true;
-      }
-    });
+  document.querySelectorAll("form").forEach(function (form) {
+    // Prevent initializing it multiple times and run it for forms
+    // that have a `form_id` field added by django-file-form.
+    if (!form.initUploadFieldsDone && form.querySelector("[name=form_id]")) {
+      init(form);
+      form.initUploadFieldsDone = true;
+    }
+  });
 
   /**
    * Initialize django-file-form for a form.
    * @param {object} form The form to initialize.
    */
   function init(form) {
-    if ($(".form__group--file").length) {
+    if (document.querySelectorAll(".form__group--file").length) {
       window.initUploadFields(form);
 
       // Hide wrapper elements for hidden inputs added by django-file-form
-      $("input[type=hidden]").closest(".form__group").hide();
+      document.querySelectorAll("input[type=hidden]").forEach(function (input) {
+        const closestFormGroup = input.closest(".form__group");
+        if (closestFormGroup) {
+          closestFormGroup.style.display = "none";
+        }
+      });
     }
   }
 });

hypha/templates/base.html

@@ -26,7 +26,7 @@
 
         <link rel="stylesheet" href="{% static 'css/tailwind-output.css' %}">
         <link rel="stylesheet" href="{% static 'css/main.css' %}">
-        <!-- Load any form css -->
+        <!-- Needed for most forms so load it here once. -->
         {{ form.media.css }}
         {% block extra_css %}{% endblock %}
         <link rel="stylesheet" href="{% static 'css/print.css' %}" media="print">
@@ -123,14 +123,16 @@
         </script>
         <!-- htmx end -->
 
-        <!-- Placeholder for the htmx modal & toast-->
+        <!-- Placeholder for the htmx modal & toast -->
         {% include "includes/_modal-placeholder.html" %}
         {% include "includes/_toast-placeholder.html" %}
 
         <!-- We still need jQuery for select2, daterangepicker and a few custom scripts. -->
         <!-- Please do not add new features that require jQuery! -->
         <script src="{% static 'js/vendor/jquery.min.js' %}"></script>
 
+        <!-- Needed for most forms so load it here once. -->
+        {{ form.media.js }}
         {% block extra_js %}{% endblock %}
 
         {% block sentry_sdk %}