emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Simple isearch concerns


From: Eli Zaretskii
Subject: Re: Simple isearch concerns
Date: Fri, 09 Apr 2021 13:50:33 +0300

> From: Juri Linkov <juri@linkov.net>
> Cc: Eli Zaretskii <eliz@gnu.org>,  spacibba@aol.com,  emacs-devel@gnu.org
> Date: Fri, 09 Apr 2021 11:37:17 +0300
> 
> >>> +@vindex isearch-allow-match-scroll
> >>
> >> IMO, the name of this variable (and the corresponding property) is not
> >> the best one.  "Scroll" is the wrong term here, and "match-scroll"
> >> doesn't help understanding what it does.  How about
> >> isearch-allow-motion-commands instead?
> >
> > I tend to agree with you, but I'm not sure.  There is already an
> > 'isearch-allow-scroll' and its corresponding 'isearch-scroll' property,
> > which changes the behavior of motion commands: it becomes possible to
> > scroll (recenter, scroll-up, scroll-down, ...) but without changing the
> > current match.  This patch does something related, but different: it
> > becomes possible to scroll among the matches.
> 
> Indeed, the name is not worse than 'isearch-allow-scroll'.  Maybe it's 
> possible
> to find a better name?

I thought I suggested a better name, right there above, in the cited
part?



reply via email to

[Prev in Thread] Current Thread [Next in Thread]