diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-24 04:01:30 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-24 04:01:30 -0500 |
| commit | 746e770ade412c8e7992b2a98441aebde000db3c (patch) | |
| tree | 81b46aa7125ebec0cc50667792a73a561c1b18f5 | |
| parent | 6ebd0314c880c4f9fc15936618ad412cef6ea309 (diff) | |
| download | dotemacs-746e770ade412c8e7992b2a98441aebde000db3c.tar.gz dotemacs-746e770ade412c8e7992b2a98441aebde000db3c.zip | |
docs(todo): close C-s isearch task — verified non-bug on Emacs 30.2
While isearch is active, overriding-terminal-local-map is isearch-mode-map, so C-s resolves to isearch-repeat-forward and the global cj/consult-line-or-repeat binding can't shadow it. isearch-mode-map already binds C-s to that default, so I left selection-framework.el unchanged.
| -rw-r--r-- | todo.org | 13 |
1 files changed, 3 insertions, 10 deletions
@@ -1653,16 +1653,9 @@ stays warm with the old VC backend. Add the resolved =file-truename= to the key, or invalidate the cache when =vc-backend= disagrees with the cached entry. -**** TODO [#B] Move =C-s= binding into =consult= =:bind= for =isearch-mode-map= :safety:solo: - -=modules/selection-framework.el:253= binds =C-s= globally to -=cj/consult-line-or-repeat=. The same module's =consult= =:bind= -block rebinds =M-s e= in =isearch-mode-map=. Once isearch is active, -pressing =C-s= should advance to the next match -- isearch's own -keymap convention -- but the global binding shadows that and exits -isearch into consult-line. Move the =C-s= binding into the consult -=:bind= block under =:map isearch-mode-map= to preserve isearch's -in-mode contract. +**** 2026-05-24 Sun @ 04:01:02 -0500 Verified C-s already advances isearch — non-bug, no change + +The premise didn't hold on Emacs 30.2. Investigated in the live daemon: while isearch is active, =overriding-terminal-local-map= is =isearch-mode-map= and the effective =C-s= resolves to =isearch-repeat-forward=, not =cj/consult-line-or-repeat=. isearch installs its own map as an overriding map, so the global =C-s= binding can't shadow it during a search. =isearch-mode-map= already binds =C-s= to =isearch-repeat-forward= by default. Adding an explicit binding to the consult =:bind= block would only duplicate that default, so I left =selection-framework.el= unchanged. **** 2026-05-16 Sat @ 02:55:14 -0500 Guarded cursor-color hook behind display-graphic-p (with daemon-mode catch) |
