GNU bug report logs - #56535
28.1; `no' and `no-ding' values for isearch-wrap-pause option (since Emacs 28.1) not working as documented

Previous Next

Package: emacs;

Reported by: Arun V <arunlalv04 <at> gmail.com>

Date: Wed, 13 Jul 2022 11:54:01 UTC

Severity: normal

Tags: moreinfo

Found in version 28.1

Fixed in version 29.0.50

Done: Juri Linkov <juri <at> linkov.net>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Arun V <arunlalv04 <at> gmail.com>
To: Juri Linkov <juri <at> linkov.net>
Cc: 56535 <at> debbugs.gnu.org, Eli Zaretskii <eliz <at> gnu.org>, visuweshm <at> gmail.com
Subject: bug#56535: 28.1; `no' and `no-ding' values for isearch-wrap-pause option (since Emacs 28.1) not working as documented
Date: Wed, 20 Jul 2022 21:46:31 +0530
Definitely ding. Lot of people do prefer visual feedback.

How about this?

  :type '(choice (const :tag "Pause before wrapping" t)
                 (const :tag "No pause before wrapping for
non-incremental search" nis-no)
                 (const :tag "No pause and no flashing for
non-incremental search" nis-no-ding)
                 (const :tag "Absolutely no pause before wrapping" no)
 ;;This will cover non-incremental searches as well
                 (const :tag "Absolutely no pause and no flashing"
no-ding)  ;;This will cover non-incremental searches as well
                 (const :tag "Disable wrapping" nil))

On Mon, 18 Jul 2022 at 01:05, Juri Linkov <juri <at> linkov.net> wrote:
>
> >>> Please try the following patch after setting isearch-wrap-pause to the
> >>> new value 'auto-repeat':
> >>>
> >> Regret the delayed response.
> >>
> >> Yes, I do believe this would be a good idea.
> >>
> >> Let me try this patch atop emacs-28.1 and get back to you.
> >
> > Thanks Juri. Your patch works.
>
> No problem with the delay.  Thanks for confirming that it works.
>
> Now we have to find a better name to describe what it really does.
> Probably it should have the prefix "no-" since it makes no pause
> like the existing two values 'no' and 'no-ding':
>
>   :type '(choice (const :tag "Pause before wrapping" t)
>                  (const :tag "No pause before wrapping" no)
>                  (const :tag "No pause and no flashing" no-ding)
>                  (const :tag "Disable wrapping" nil))
>
> This raises the question: should it ding and flash on wrapping?
> Or two new separate values are needed: one to ding when wrapping
> on typing a character, and another not to ding?  To ding, or not to ding?



-- 
--
Thanks and regards,
Arunlal Venugopal




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

Previous Next


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