[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#66394: 29.1; Make register-read-with-preview more useful
From: |
Stefan Monnier |
Subject: |
bug#66394: 29.1; Make register-read-with-preview more useful |
Date: |
Sat, 16 Dec 2023 10:31:50 -0500 |
User-agent: |
Gnus/5.13 (Gnus v5.13) |
> It should be possible to use post-command-hook, I didn't use it because
> it makes harder the communication between the minibuffer and the preview
> buffer.
The patch below seems to work for my extremely limited testing.
Stefan
diff --git a/lisp/register.el b/lisp/register.el
index fa4bbcf483f..31466a42b0a 100644
--- a/lisp/register.el
+++ b/lisp/register.el
@@ -328,7 +328,7 @@ register-read-with-preview
m))
(data (register-command-info this-command))
(enable-recursive-minibuffers t)
- types msg result timer act win strs smatch)
+ types msg result act win strs smatch)
(if data
(setq types (register-preview-info-types data)
msg (register-preview-info-msg data)
@@ -360,9 +360,7 @@ register-read-with-preview
(progn
(minibuffer-with-setup-hook
(lambda ()
- (setq timer
- (run-with-idle-timer
- 0.01 'repeat
+ (add-hook 'post-command-hook
(lambda ()
(with-selected-window (minibuffer-window)
(let ((input (minibuffer-contents)))
@@ -408,13 +406,13 @@ register-read-with-preview
;; jump, don't ask for confirmation
;; and exit immediately (bug#66394).
(setq result pat)
- (exit-minibuffer)))))))))
+ (exit-minibuffer))))))
+ nil 'local))
(setq result (read-from-minibuffer
prompt nil map nil nil
(register-preview-get-defaults act))))
(cl-assert (and result (not (string= result "")))
nil "No register specified")
(string-to-char result))
- (when timer (cancel-timer timer))
(let ((w (get-buffer-window buf)))
(and (window-live-p w) (delete-window w)))
(and (get-buffer buf) (kill-buffer buf)))))
- bug#66394: 29.1; Make register-read-with-preview more useful, (continued)
- bug#66394: 29.1; Make register-read-with-preview more useful, Stefan Kangas, 2023/12/14
- bug#66394: 29.1; Make register-read-with-preview more useful, Thierry Volpiatto, 2023/12/15
- bug#66394: 29.1; Make register-read-with-preview more useful, Stefan Monnier, 2023/12/15
- bug#66394: 29.1; Make register-read-with-preview more useful, Thierry Volpiatto, 2023/12/15
- bug#66394: 29.1; Make register-read-with-preview more useful, Stefan Monnier, 2023/12/15
- bug#66394: 29.1; Make register-read-with-preview more useful, Thierry Volpiatto, 2023/12/16
- bug#66394: 29.1; Make register-read-with-preview more useful,
Stefan Monnier <=
- bug#66394: 29.1; Make register-read-with-preview more useful, Thierry Volpiatto, 2023/12/16
- bug#66394: 29.1; Make register-read-with-preview more useful, Stefan Monnier, 2023/12/17
- bug#66394: 29.1; Make register-read-with-preview more useful, Thierry Volpiatto, 2023/12/18
- bug#66394: 29.1; Make register-read-with-preview more useful, Stefan Monnier, 2023/12/18
- bug#66394: 29.1; Make register-read-with-preview more useful, Thierry Volpiatto, 2023/12/18
- bug#66394: 29.1; Make register-read-with-preview more useful, Dmitry Gutov, 2023/12/18
- bug#66394: 29.1; Make register-read-with-preview more useful, Thierry Volpiatto, 2023/12/18
- bug#66394: 29.1; Make register-read-with-preview more useful, Thierry Volpiatto, 2023/12/19
- bug#66394: 29.1; Make register-read-with-preview more useful, Thierry Volpiatto, 2023/12/19
- bug#66394: 29.1; Make register-read-with-preview more useful, Eli Zaretskii, 2023/12/20