bugGNU nano - Bugs: bug #47127, replacing shows each match on the...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #47127: replacing shows each match on the bottom line

Submitter:  Benno Schulenberg <bens>
Submitted:  Fri 12 Feb 2016 08:49:51 PM UTC
   
 
Severity:  4 - Important Status:  Fixed
Assigned to:  bens Open/Closed:  Closed

Sun 14 Feb 2016 10:21:20 AM UTC, comment #3: 

Fixed in SVN, r5641 for replacing and r5642 for spell fixing.

Will post a new bug to address the more general issue.


Benno Schulenberg <bens>
Group administrator
Sat 13 Feb 2016 08:37:30 PM UTC, comment #2: 

Attached patch fixes it for Replacing.  For the internal speller something similar can probably be done, but am too tired now.

(file #36327)

Benno Schulenberg <bens>
Group administrator
Sat 13 Feb 2016 11:22:01 AM UTC, comment #1: 

This is a more wide-spread problem: also the internal spell checker will also put each currently off-screen match on the bottom line, typing M-/ (to last line) will put the cursor on the bottom line whereas Pico (and nano before 2.4.1) will center it.

This latter behavior can be argued for and against, but not centering matches... I don't like it.

It is all caused by the fix for bug #42654.  I think things should instead behave like it was suggested there: when a one-line scroll can reach the next match or word or whatever, it should scroll just one line; if it requires scrolling more lines, the line that is jumped to should be centered.

Benno Schulenberg <bens>
Group administrator
Fri 12 Feb 2016 08:49:51 PM UTC, original submission:  

Since nano-2.4.1, replacing a string and answering for each occurrence separately, will show each match that is currently off-screen on the bottom line.  Just searching (^W) for matches, will show each off-screen match on the center line.  Replacing should work the same -- like it did before 2.4.1.

Benno Schulenberg <bens>
Group administrator

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by bens (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2016-02-27 bens Open/ClosedOpen Closed
    2016-02-14 bens StatusIn Progress Fixed
    2016-02-13 bens Attached File- Added make-replace-behave-as-whereis.patch, #36327
        StatusNone In Progress

    Back to the top

    Powered by Savane 3.13-4448.
    Corresponding source code