GNU bug report logs - #62709
29.0.60; quail-minibuffer-message prevents set-mark-command from activating region in minibuffer

Previous Next

Package: emacs;

Reported by: Visuwesh <visuweshm <at> gmail.com>

Date: Fri, 7 Apr 2023 10:56:01 UTC

Severity: normal

Found in version 29.0.60

Done: Eli Zaretskii <eliz <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Eli Zaretskii <eliz <at> gnu.org>
To: Visuwesh <visuweshm <at> gmail.com>
Cc: 62709 <at> debbugs.gnu.org
Subject: bug#62709: 29.0.60; quail-minibuffer-message prevents set-mark-command from activating region in minibuffer
Date: Fri, 07 Apr 2023 17:57:02 +0300
> From: Visuwesh <visuweshm <at> gmail.com>
> Cc: 62709 <at> debbugs.gnu.org
> Date: Fri, 07 Apr 2023 20:09:13 +0530
> 
> [வெள்ளி ஏப்ரல் 07, 2023] Visuwesh wrote:
> 
> >> I'd prefer to solve the problem that prevents the region from becoming
> >> active, without changing how Quail shows its messages.  Can you try
> >> finding such a solution, 
> >
> > One solution would be to use an overlay to show the guidance string,
> > like what `minibuffer-depth-mode' does.
> 
> I came close to solving the issue with an overlay but I hit a roadblock
> with the cursor property.

Thanks, but I think this is an unnecessary complication.

Does the patch below fix the problems you saw, without introducing any
regressions?

diff --git a/lisp/international/quail.el b/lisp/international/quail.el
index 2ffe339..317ea84 100644
--- a/lisp/international/quail.el
+++ b/lisp/international/quail.el
@@ -1995,7 +1995,8 @@ quail-require-guidance-buf
 (defun quail-minibuffer-message (string)
   (message nil)
   (let ((point-max (point-max))
-	(inhibit-quit t))
+	(inhibit-quit t)
+        (deactivate-mark nil))
     (save-excursion
       (goto-char point-max)
       (insert string))




This bug report was last modified 2 years and 106 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.