14173 - Autoscrolling while selecting is awkward

Status: VERIFIED FIXED

(Core :: Editor, defect, P3)

Description

[Phil Peterson]

I've been trying to use seamonkey for bugzilla work, and one of the major
problems I'm having is that attempting to scroll while selecting text is
awkward.

It seems to only recognize the mouse being out of the clipping area in the X
direction, not in the Y direction. Also, it seems to require that you continue
to wave the mouse back and forth to get scrolling behavior. I'd expect it to
allow you to keep the mouse stationary and scroll automatically.

Comment 1

[buster]

I think this is just a reflection of the mouse auto-scrolling code being only
partially implemented.  Kin, if you have other bugs that cover this, dup this
one, otherwise use it as a feature placeholder.

Comment 2

[kinmoz]

Accepting bug. Marking M11.

I am currently working on getting autoscrolling working.

I'm not exactly sure how phil is getting things to autoscroll since
autoscrolling really isn't implemented yet.

In any case part of the autoscrolling feature will use timers that will fire off
and scroll the view when the mouse isn't moving.

Comment 3

[kinmoz]

Changing milestone of all Selection Scrolling bugs to M12.

Comment 4

[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 5

[sujay]

verified in 10/13 build.