#51 Wrap position restore

closed
Don HO
None
8
2008-03-15
2008-01-30
Harry
No

Patch alters behaviour so when wrapping is enabled and the user switches from one tab to another and back, the position is correctly restored to where the user left off.

Also, slight alteration in Notepad_plus.cpp with the fullscreen function so ZORDER of windows shouldnt be altered when switching back to window (usually bad behaviour shouldnt occur but just in case..).

This bug has been around for way too long ;)

Main difference is that some more scrolling occurs AFTER the screen has been painted etc so Scintilla can do its layout, as is described in its documentation.

Discussion

  • Harry

    Harry - 2008-01-30

    Archive containing patch files

     
  • Garthex

    Garthex - 2008-01-30

    Logged In: YES
    user_id=1943765
    Originator: NO

    When does this occur? The cursor is in the same position as before when switching tabs with line wrapping enabled, for me.

     
  • Harry

    Harry - 2008-01-30

    Logged In: YES
    user_id=1831293
    Originator: YES

    The cursor is put on the right location, but the view of the lines tends to shift somewhat.

    Btw, theres still a problem with the patch, when you resize the Notepad++ window, it still may displace the view somewhat, although it should be less than it was before (shrink the window and it may not scroll far enough, enlarge it and it will scroll too far).
    Should still perform better than the current behaviour though ;).
    (The problem is is that the actual position in the topleft cannot be determined properly, online the visible line. Therefore, this method attempts to recover _that_ line by number, although the actual line that contains the position to be recovered may have moved because of resizing, this shouldnt happen too often though).

    The problem gets more noticable when there are many very long lines so many extra wraps are made.

     
  • Nobody/Anonymous

    Logged In: NO

    Please do remember that this problem also occurres with find tool positioning.. if the current patch doesn't already correct this.

     
  • Don HO

    Don HO - 2008-02-03
    • priority: 5 --> 8
    • assigned_to: nobody --> donho
     
  • Don HO

    Don HO - 2008-03-15
    • status: open --> closed
     

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks