Bugs item #1663839, was opened at 2007-02-19 13:05
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=102439&aid=1663839&group_id=2439

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Scintilla
Group: Bug
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Nobody/Anonymous (nobody)
Summary: stuttered page down doesn't work if lines are wrapped

Initial Comment:
The "stuttered page down" command doesn't work if the current page contains 
wrapped lines - on the first Page Down, the cursor will go to the bottom line 
on the page, and then it won't budge on subsequent Page Down keys.

The problem is that the calculation of the bottom stuttered line in 
Editor::PageMove() assumes that one display line equals one document line, 
which is wrong if there are any wrapped lines on the page.  This is at line 
4023 in Editor.cxx - changing the line as follows seems to fix it:

        int bottomStutterLine =
                pdoc->LineFromPosition(PositionFromLocation(
                Point(lastXChosen, direction * vs.lineHeight * 
LinesToScroll())))
                - caretYSlop - 1;

The idea here is that we calculate the bottom stuttered line as the actual 
document line at the bottom of the page.  This takes into account the current 
wrapping situation, so it gets the right line.

Contact me at [EMAIL PROTECTED] with any questions.


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=102439&aid=1663839&group_id=2439
_______________________________________________
Scintilla-interest mailing list
[email protected]
http://mailman.lyra.org/mailman/listinfo/scintilla-interest

Reply via email to