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

 
 

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

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

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

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>
Project AdministratorIn charge of this item.
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>
Project AdministratorIn charge of this item.
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>
Project AdministratorIn charge of this item.
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>
Project AdministratorIn charge of this item.

 

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

Attach File(s):
   
   
Comment:
   

Attached Files

 

Depends on the following items: None found

Items that depend on this one: None found

 

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

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sat 27 Feb 2016 03:04:30 PM UTCbensOpen/ClosedOpen=>Closed
    Sun 14 Feb 2016 10:21:20 AM UTCbensStatusIn Progress=>Fixed
    Sat 13 Feb 2016 08:37:30 PM UTCbensAttached File-=>Added make-replace-behave-as-whereis.patch, #36327
      StatusNone=>In Progress

    Back to the top


    Powered by Savane 3.1-cleanup1