text wrapping not moving scrollbar in proper direction

VERIFIED FIXED in M10

Status

()

Core
Editor
P3
normal
VERIFIED FIXED
19 years ago
16 years ago

People

(Reporter: sujay, Assigned: kinmoz)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

19 years ago
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.
(Reporter)

Comment 1

19 years ago
forgot to mention that I noticed this when comparing to 4.x.

Updated

19 years ago
Assignee: sfraser → mjudge

Comment 2

19 years ago
ScrollSelectionIntoView should work horizontally, as wel as vertically.
Reassigning to mjudge.

Updated

19 years ago
Target Milestone: M10

Comment 3

19 years ago
set to milestone M10
(Assignee)

Updated

19 years ago
Assignee: mjudge → kin
(Assignee)

Comment 4

19 years ago
Reassigning to kin@netscape.com. I have a fix for this.
(Assignee)

Updated

19 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 5

19 years ago
accepting bug
(Assignee)

Updated

19 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED
(Assignee)

Comment 6

19 years ago
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().
(Reporter)

Updated

19 years ago
Status: RESOLVED → VERIFIED
(Reporter)

Comment 7

19 years ago
verified in 9/2 build.
You need to log in before you can comment on or make changes to this bug.