8015 - full page text drag-selection should autoscroll to end of page

Status: VERIFIED FIXED

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

Description

[sujay]

using 6/11 build of apprunner on all platforms:

1) launch apprunner
2) launch editor
3) try selecting text using the mouse all the way down to the bottom.

Notice it doesn't keep going to the end of the document. It should keep
scrolling to the end of doc with all the text selected.

Comparing to 4.x Composer. works fine there...

Comment 1

[Kathleen :Brade]

updated summary
note: also should autoscroll to top of page if user starts at the bottom

Comment 2

[Kathleen :Brade]

set to M11
I assume this is also a bug in the browser; sujay, can you confirm?

Comment 3

[sujay]

yes same problem in browser also..

Comment 4

[kinmoz]

Reassigning to [e-mail]

Comment 5

[kinmoz]

accepting bug

Comment 6

[kinmoz]

Changing milestone of all Selection Scrolling bugs to M12.

Comment 7

[kinmoz]

Fix checked in:

    layout/base/public/nsIFrame.h  revision 3.129
        - Added GetContentAndOffsetsFromPoint() method. This
          method will eventually replace GetPosition().

    layout/base/public/nsIFrameSelection.h  revision 1.18
        - Added HandleDrag() and Start/StopAutoScrollTimer() methods.

    layout/base/src/nsRangeList.cpp  revision 1.145
        - Added implementation for HandleDrag() and
          Start/StopAutoScrollTimer() methods.

    layout/html/base/src/nsBRFrame.cpp  revision 1.21
        - Added implementation for GetContentAndOffsetsFromPoint().

    layout/html/base/src/nsFrame.cpp  revision 3.157
    layout/html/base/src/nsFrame.h    revision 3.89
        - Added implementation for GetContentAndOffsetsFromPoint().
        - Added GetClosestViewForFrame() utility method.
        - Modified HandleEvent() to call HandleRelease() which now
          stops the auto scrolling timers.
        - Modified HandleDrag() to call nsIFrameSelection's HandleDrag()
          and Start/StopAutoScrollTimer() methods.

    layout/html/base/src/nsTextFrame.cpp  revision 1.176
        - Added implementation for GetContentAndOffsetsFromPoint().

Comment 8

[sujay]

verified in 10/13 build. excellent work!