Closed Bug 7153 Opened 25 years ago Closed 25 years ago

text wrapping not moving scrollbar in proper direction

Categories

(Core :: DOM: Editor, defect, P3)

defect

Tracking

()

VERIFIED FIXED

People

(Reporter: sujay, Assigned: kinmoz)

Details

using 5/26 build of apprunner on all platforms 1) launch apprunner 2) launch editor from task menu 3) insert cursor directly after a character(this way you're adding chars to a word) 4) now type garbage and watch the scrollbar...keep typing garbage. The scrollbar is moving in the wrong direction. We should be following the cursor. We're not. all platforms.
forgot to mention that I noticed this when comparing to 4.x.
Assignee: sfraser → mjudge
ScrollSelectionIntoView should work horizontally, as wel as vertically. Reassigning to mjudge.
Target Milestone: M10
set to milestone M10
Assignee: mjudge → kin
Reassigning to kin@netscape.com. I have a fix for this.
Status: NEW → ASSIGNED
accepting bug
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Fix for bugs #7153, #10673, #12066, #12067, and #12793. editor/base/nsEditorEventListeners.cpp revision 1.89 editor/base/nsEditorEventListeners.h revision 1.115 - Added code to scroll the selection into view after processing key events. - Commented out the hack that redraws the entire view when the focus is gained and lost. Replaced the hack code with calls to RepaintSelection(). layout/base/public/nsIFrameSelection.h revision 1.115 layout/base/public/nsIPresShell.h revision 3.46 layout/html/base/src/nsPresShell.cpp revision 3.167 - Added ScrollSelectionIntoView() and RepaintSelection() methods. layout/base/src/nsRangeList.cpp revision 1.123 - Added implementation for ScrollSelectionIntoView() and RepaintSelection(). - Check for NULL primary frame in GetFocusNodeRect().
Status: RESOLVED → VERIFIED
verified in 9/2 build.
You need to log in before you can comment on or make changes to this bug.