diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-10 12:39:44 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-10 12:39:44 -0500 |
| commit | 5eccbf7bd3c9780eee6170839dbf3224a32e9ef3 (patch) | |
| tree | 524513d6882e87bb39921c4e0c4c79cc9f41ca16 /tests | |
| parent | c56a638192f3b6aff13c26e34ce78db0d26de6fc (diff) | |
| download | dotemacs-5eccbf7bd3c9780eee6170839dbf3224a32e9ef3.tar.gz dotemacs-5eccbf7bd3c9780eee6170839dbf3224a32e9ef3.zip | |
refactor(vterm): move vterm prefix to C-; x and add prompt nav
The personal vterm map was on `C-; V'. The capital V costs a Shift on every keystroke into the menu, which adds up for the daily `C-; V c' / `C-; V C' bindings. Move the prefix to lowercase `C-; x' -- free, no Shift, faintly mnemonic (xterm/execute). The lowercase `C-; v' stays the version-control menu.
Wire `vterm-next-prompt' and `vterm-previous-prompt' into the menu so they're reachable everywhere, not only inside vterm-copy-mode-map. Lowercase `n' and `p' match Emacs's idiom for next/previous; bump "new vterm" up to capital `N' for the rare new-buffer case.
Drop the `<pause>' binding for `vterm-copy-mode' from `vterm-mode-map'. Modern keyboards rarely have a Pause key and `C-; x c' is the canonical entry now.
Update which-key labels and tests; `test-vterm-keymap-includes-history-and-copy-bindings' now asserts the new prefix, and two new tests cover prompt-nav bindings and the dropped `<pause>' binding.
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/test-vterm-tmux-history.el | 23 |
1 files changed, 18 insertions, 5 deletions
diff --git a/tests/test-vterm-tmux-history.el b/tests/test-vterm-tmux-history.el index 754ddc9c..f5263df5 100644 --- a/tests/test-vterm-tmux-history.el +++ b/tests/test-vterm-tmux-history.el @@ -120,14 +120,27 @@ RESPONSES is an alist of (ARGS EXIT-CODE OUTPUT)." (ert-deftest test-vterm-keymap-includes-history-and-copy-bindings () "Normal: personal vterm map owns the high-level vterm UX commands." (should (member "C-;" vterm-keymap-exceptions)) - (should-not (eq (keymap-lookup cj/custom-keymap "v c") #'vterm-copy-mode)) - (should (eq (keymap-lookup cj/custom-keymap "V C") #'cj/vterm-tmux-history)) - (should (eq (keymap-lookup cj/custom-keymap "V c") #'vterm-copy-mode)) + (should-not (eq (keymap-lookup cj/custom-keymap "X c") #'vterm-copy-mode)) + (should (eq (keymap-lookup cj/custom-keymap "x C") #'cj/vterm-tmux-history)) + (should (eq (keymap-lookup cj/custom-keymap "x c") #'vterm-copy-mode)) (should (equal (keymap-lookup vterm-mode-map "C-;") cj/custom-keymap)) - (should (eq (keymap-lookup vterm-mode-map "C-; V C") #'cj/vterm-tmux-history)) - (should (eq (keymap-lookup vterm-mode-map "C-; V c") #'vterm-copy-mode)) + (should (eq (keymap-lookup vterm-mode-map "C-; x C") #'cj/vterm-tmux-history)) + (should (eq (keymap-lookup vterm-mode-map "C-; x c") #'vterm-copy-mode)) (should-not (keymap-lookup vterm-mode-map "C-c C-t"))) +(ert-deftest test-vterm-keymap-prompt-navigation () + "Normal: n/p navigate prompts, capital N creates a new vterm buffer." + (should (eq (keymap-lookup cj/custom-keymap "x n") #'vterm-next-prompt)) + (should (eq (keymap-lookup cj/custom-keymap "x p") #'vterm-previous-prompt)) + (should (eq (keymap-lookup cj/custom-keymap "x N") #'vterm))) + +(ert-deftest test-vterm-pause-not-bound-to-copy-mode () + "Normal: <pause> is no longer wired as a vterm-copy-mode entry point. +The personal `C-; x c' binding is the canonical entry; <pause> is rare on +modern keyboards and was redundant." + (let ((binding (keymap-lookup vterm-mode-map "<pause>"))) + (should-not (eq binding #'vterm-copy-mode)))) + (ert-deftest test-vterm-copy-mode-cancel-keys () "Normal: copy mode has explicit copy and no-copy exits." (should (eq (keymap-lookup vterm-copy-mode-map "C-g") |
