Rewrite the save shortcut JS without jQuery

Author: matthiask

.pre-commit-config.yaml

@@ -14,12 +14,12 @@ repos:
       - id: mixed-line-ending
       - id: trailing-whitespace
   - repo: https://github.com/adamchainz/django-upgrade
-    rev: 1.24.0
+    rev: 1.25.0
     hooks:
       - id: django-upgrade
         args: [--target-version, "3.2"]
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: "v0.11.5"
+    rev: "v0.12.0"
     hooks:
       - id: ruff
         args: [--unsafe-fixes]
@@ -30,7 +30,7 @@ repos:
       - id: biome-check
         args: [--unsafe]
   - repo: https://github.com/tox-dev/pyproject-fmt
-    rev: v2.5.1
+    rev: v2.6.0
     hooks:
       - id: pyproject-fmt
   - repo: https://github.com/abravalheri/validate-pyproject

CHANGELOG.rst

@@ -11,6 +11,7 @@ Next version
 - Prepared for the new object-based ``Script`` by moving from django-js-asset's
   ``JS`` to a script without attributes, and moved the configuration into an
   inline JSON blob.
+- Rewrote ``content_editor/save_shortcut.js`` without jQuery.
 
 
 7.2 (2025-01-27)

content_editor/static/content_editor/save_shortcut.js

@@ -1,9 +1,11 @@
-/* global django */
-django.jQuery(($) => {
-  $(document).keydown(function handleKeys(event) {
-    if (event.which === 83 && (event.metaKey || event.ctrlKey)) {
-      $("form input[name=_continue]").click()
-      return false
-    }
-  })
+window.addEventListener("keydown", (e) => {
+  let el
+  if (
+    e.key === "s" &&
+    (e.metaKey || e.ctrlKey) &&
+    (el = document.querySelector("form input[name=_continue]"))
+  ) {
+    e.preventDefault()
+    el.click()
+  }
 })