Remove jQuery loading by default (#252)

- Sill used as a peer dependency of Bootstrap
- Core jQuery dependency removed though
- Fixes #250

Co-authored-by: LB Johnston <[email protected]>

Author: lb-

package-lock.json

@@ -21,8 +21,7 @@
     "@wagtail/eslint-config-wagtail": "^0.4.0",
     "autocompleter": "^6.0.3",
     "bootstrap": "^4.4.1",
-    "eslint": "^8.16.0",
-    "jquery": "^3.5.1"
+    "eslint": "^8.16.0"
   },
   "scripts": {
     "start": "npm run watch",

package.json

@@ -184,7 +184,7 @@
         <script type="text/javascript" src="{{ pathto('_static/searchtools.js', 1) }}"></script>
         <script type="text/javascript" src="{{ pathto('_static/language_data.js', 1) }}"></script>
         <script type="text/javascript">
-            jQuery(function() { Search.loadIndex("{{ pathto('searchindex.js', 1) }}"); });
+            document.addEventListener('DOMContentLoaded', function() { Search.loadIndex("{{ pathto('searchindex.js', 1) }}"); });
         </script>
         <script type="text/javascript" id="searchindexloader"></script>
     {% endblock -%}

sphinx_wagtail_theme/layout.html

@@ -13,9 +13,6 @@ module.exports = {
     publicPath: "",
     filename: "[name].js",
   },
-  externals: {
-    jquery: "window.$",
-  },
   module: {
     rules: [
       {