aboutsummaryrefslogtreecommitdiff
path: root/tests
diff options
context:
space:
mode:
authorCraig Jennings <c@cjennings.net>2026-06-13 00:56:38 -0500
committerCraig Jennings <c@cjennings.net>2026-06-13 00:56:38 -0500
commita631938c95ee7974a1d5489ad3aba713c4f0a0d5 (patch)
treeb0e9fe7b88fa3f0707f4787429cbf2554a2a91af /tests
parent2cc54965d614151c8d8df7c0c857cbcd4aedf1af (diff)
downloaddotemacs-a631938c95ee7974a1d5489ad3aba713c4f0a0d5.tar.gz
dotemacs-a631938c95ee7974a1d5489ad3aba713c4f0a0d5.zip
fix(keys): revert 4a1ecf64, restore the M-S- launcher bindings
4a1ecf64 rebound eww/elfeed/calibredb from M-S-e/r/b to the uppercase Meta events M-E/M-R/M-B, on the theory that the keyboard emits the uppercase event and the lowercase-shift spec never matched. It was wrong. keyboard-compat.el installs a key-translation-map entry (M-E -> M-S-e, etc.) in GUI frames, so the original M-S- bindings did reach the launchers. Binding M-E directly while that translation stayed in place rewrote the keypress to the now-unbound M-S-e, breaking all three in GUI on the next restart. The audit and the review missed it because they checked key-binding, which ignores key-translation-map, and the running daemon still held the pre-fix bindings as stale state. This restores M-S-e/r/b, deletes the key-binding-only test that certified the broken config, and reclassifies the task as not-a-bug. The real problem, that these chords are dead outside GUI, is the subject of the keybinding-console-safety spec.
Diffstat (limited to 'tests')
-rw-r--r--tests/test-launcher-meta-shift-keys.el32
1 files changed, 0 insertions, 32 deletions
diff --git a/tests/test-launcher-meta-shift-keys.el b/tests/test-launcher-meta-shift-keys.el
deleted file mode 100644
index 574a8d68e..000000000
--- a/tests/test-launcher-meta-shift-keys.el
+++ /dev/null
@@ -1,32 +0,0 @@
-;;; test-launcher-meta-shift-keys.el --- Meta+Shift launcher keys reach their commands -*- lexical-binding: t; -*-
-
-;;; Commentary:
-;; Meta+Shift+<letter> emits the uppercase event (M-E, M-R, M-B), so a
-;; binding written as "M-S-e" on a lowercase letter is never reached by
-;; the keychord. These launchers must be bound under the uppercase
-;; event the keyboard actually produces.
-
-;;; Code:
-
-(require 'ert)
-(require 'testutil-general)
-
-(add-to-list 'load-path (expand-file-name "modules" user-emacs-directory))
-(require 'eww-config)
-(require 'elfeed-config)
-(require 'calibredb-epub-config)
-
-(ert-deftest test-launcher-meta-shift-e-opens-eww ()
- "Normal: M-E (Meta+Shift+e) is bound to `eww'."
- (should (eq (key-binding (kbd "M-E")) 'eww)))
-
-(ert-deftest test-launcher-meta-shift-r-opens-elfeed ()
- "Normal: M-R (Meta+Shift+r) is bound to `cj/elfeed-open'."
- (should (eq (key-binding (kbd "M-R")) 'cj/elfeed-open)))
-
-(ert-deftest test-launcher-meta-shift-b-opens-calibredb ()
- "Normal: M-B (Meta+Shift+b) is bound to `calibredb'."
- (should (eq (key-binding (kbd "M-B")) 'calibredb)))
-
-(provide 'test-launcher-meta-shift-keys)
-;;; test-launcher-meta-shift-keys.el ends here