GNU bug report logs - #74218
[PATCH] Ask confirmation before sending region to search engine.

Previous Next

Package: emacs;

Reported by: Fabio Natali <me <at> fabionatali.com>

Date: Wed, 6 Nov 2024 00:57:02 UTC

Severity: normal

Tags: patch

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: Eshel Yaron <me <at> eshelyaron.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 74218 <at> debbugs.gnu.org, stefankangas <at> gmail.com, me <at> fabionatali.com
Subject: bug#74218: [PATCH] Ask confirmation before sending region to search engine.
Date: Thu, 07 Nov 2024 10:12:53 +0100
Eli Zaretskii <eliz <at> gnu.org> writes:

>> From: Eshel Yaron <me <at> eshelyaron.com>
>> Cc: Fabio Natali <me <at> fabionatali.com>,  Eli Zaretskii <eliz <at> gnu.org>,
>>   74218 <at> debbugs.gnu.org
>> Date: Thu, 07 Nov 2024 09:42:29 +0100
>> 
>> I too agree that it's a good idea to optionally require confirmation.
>> However, I suspect that a yes/no question is not the best interface in
>> this case.  Instead, it's better to simply prepopulate the minibuffer
>> with the contents of the region.  Then you confirm with RET and cancel
>> with C-g.  In addition, this lets you examine and edit your input.
>
> Why copy the region into the mini-window when it is already shown in
> the current buffer's window?  By default, it will be highlighted, but
> if not (e.g., transient-mark-mode was disabled), we could forcibly
> highlight it.  Why is that not enough?

While point is always visible, mark can be out of view, so the region
need not be fully visible in the selected window.  But more importantly,
using the minibuffer provides a smoother and more consistent UX compared
to an additional yes/no question, IMO.

> Copying stuff into the minibuffer has the disadvantage of resizing the
> mini-window, and then it could hit the limits on such resizes, which
> will prevent the user from seeing large portions of the text, if the
> region is large.
>
> Also, does anyone have an opinion about asking for confirmation only
> for regions that are large enough?  E.g., when the region is a single
> word, do we want to ask for confirmation anyway?

I think it makes sense to have an option that is sensitive to the size
of the region, although personally I'd probably stick to "always ask",
especially if the prompt for confirmation isn't too obtrusive.




This bug report was last modified 209 days ago.

Previous Next


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