GNU bug report logs - #15839
24.3.50; `isearch-allow-scroll': be able to scroll point off screen temporarily

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Drew Adams <drew.adams <at> oracle.com>
Subject: bug#15839: closed (Re: bug#15839: 24.3.50; `isearch-allow-scroll':
 be able to scroll point off screen temporarily)
Date: Tue, 04 Dec 2018 00:43:02 +0000

[Message part 1 (text/plain, inline)]

Your bug report

#15839: 24.3.50; `isearch-allow-scroll': be able to scroll point off screen temporarily

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 15839 <at> debbugs.gnu.org.

-- 
15839: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=15839
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Juri Linkov <juri <at> linkov.net>
To: Drew Adams <drew.adams <at> oracle.com>
Cc: 15839-done <at> debbugs.gnu.org
Subject: Re: bug#15839: 24.3.50;
 `isearch-allow-scroll': be able to scroll point off screen temporarily
Date: Tue, 04 Dec 2018 02:29:42 +0200

> But I don't really need to understand.  I reviewed
> what you wrote about that only because you included
> it in a reply to this (other) bug report.  Ignore my
> feedback about `*-allow-shift-selection' if you like
> - no problem.

Thank you for your feedback, all your remarks were taken into consideration,
committed to master, and this enhancement request closed.

[Message part 3 (message/rfc822, inline)]

From: Drew Adams <drew.adams <at> oracle.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 24.3.50; `isearch-allow-scroll': be able to scroll point off screen
 temporarily
Date: Fri, 8 Nov 2013 15:02:42 -0800 (PST)

1. Non-nil `isearch-allow-scroll' lets you use a scroll command
(e.g. `C-v') without exiting Isearch.  Unfortunately, this is coupled
with the hard-coded behavior that you cannot scroll far enough in either
direction that point would be moved off screen.

That restriction is general for Emacs, and it generally makes sense.
It does not necessarily make sense during Isearch, however.  Why?

Because the search position is one thing, and it is not lost.  What you
might want to look at temporarily is another thing.

It's a bit like using `C-SPC' in a buffer, scrolling up a couple of
screenfuls to look at something, and then using `C-u C-SPC' to return.
But in Isearch there is no need for `C-SPC' or `C-u C-SPC': the search
position is recorded.  Search resumes from that same position, no
matter how far away one might have scrolled.

The enhancement request is to let users choose whether non-nil
`isearch-allow-scroll' should limit you to scrolling only enough to keep
point in the window or should not limit you.  This could be done by
recognizing different non-nil values.

2. What's more, the lazy highlighting of search hits is even more
limited currently.  When you scroll to the current limit, there can be
lots of search hits that are not highlighted.

One of the reasons for scrolling is to see what search hits lie beyond
the currently shown text.  This is already partly defeated, in that
even when you scroll some of the hits are not highlighted.  This seems
more like a bug, but if the enhancement of #1 is done then it should
(it needs to) take care of #2 as well: Whatever text is shown should
have all of its search hits highlighted with lazy highlighting.

In GNU Emacs 24.3.50.1 (i686-pc-mingw32)
 of 2013-10-19 on LEG570
Bzr revision: 114715 rgm <at> gnu.org-20131019023520-s8mwtib7xcx9e05w
Windowing system distributor `Microsoft Corp.', version 6.1.7601
Configured using:
 `configure --enable-checking 'CFLAGS=-O0 -g3' CPPFLAGS=-DGLYPH_DEBUG=1'

Previous Next

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