Closed
Bug 17989
Opened 25 years ago
Closed 25 years ago
[DOGFOOD] Selection AutoScrolling broken when using GFX Scrollbars
Categories
(Core :: DOM: Selection, defect, P1)
Core
DOM: Selection
Tracking
()
VERIFIED
FIXED
M12
People
(Reporter: kinmoz, Assigned: kinmoz)
References
Details
(Whiteboard: [PDT+] [by 11/18])
Selection AutoScrolling doesn't work when using Gfx Scrollbars.
To reproduce:
1. Edit your prefs.js file to include the following line.
user_pref("nglayout.widget.gfxscrollbars", true);
2. Start the browser and load a page that is larger than the viewable area. This
will force the scrollbars in the content area to appear.
3. Click and hold the mouse over some text and then drag out of the window.
The content area should auto scroll to show you the content that is in the
direction the mouse is moving, but it doesn't.
Comment 2•25 years ago
|
||
Duplicate of 17830?
Updated•25 years ago
|
Priority: P3 → P1
Comment 4•25 years ago
|
||
bumping up priority since it's a PDT+
Comment 7•25 years ago
|
||
[note that 17830 contains additional details not present in this bug report.]
Updated•25 years ago
|
Whiteboard: [PDT+] → [PDT+] [by 11/15]
Comment 8•25 years ago
|
||
this has been investigated and Kin has an idea on how to resolve it, he may or
may not need mjudge to help resolve it.
Comment 9•25 years ago
|
||
Additional details:
Selection Autoscrolling with gfx-scrollbars actually seems to snap back to the
beginning of the page instead of autoscrolling.
Steps to reproduce:
1. Load a webpage that will be several screenfuls - http://news.bbc.co.uk/
will do fine.
2. Scroll down or [PgDn] more than one screenful.
3. Attempt to invoke selection autoscrolling by click-dragging down beyond
the edge of the canvas.
Actual results:
The page will snap back to the top of the first screenful. If you scroll down
to look, less will be selected than was selected just before the canvas bottom
was reached.
This result looks much like what happens when dragging a scrollbar slider down
if the pointer wanders left or right out of the scrollbar, except here the
jump was not back to the origin, but to the top of the page.
Expected result:
Autoscrolling selection of text.
Does not work correctly with:
1999-11-09-11-M11 nightly binary on Windows NT 4.0sp3.
1999-11-10-09-M12, 1999-10-18-08-M11, and other builds in between.
Additional Information:
1. The reporter of bug 17830 says that the same snap-back effect happens
horizontally - confirmed.
2. Selection autoscrolling *upwards* does work correctly but slowly,
pixel-by-pixel, not line-by-line of text.
Assignee | ||
Comment 10•25 years ago
|
||
I have a fix for this in my tree. I'll probably check it in on 11/18/1999 after
I've had it code reviewed.
Whiteboard: [PDT+] [by 11/18] → [PDT+] [by 11/18] Currently waiting on a code review from evaughan@netscape.com
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Whiteboard: [PDT+] [by 11/18] Currently waiting on a code review from evaughan@netscape.com → [PDT+] [by 11/18]
Assignee | ||
Comment 11•25 years ago
|
||
Fix is checked in and should appear in the 11/19/99 builds.
layout/base/src/nsRangeList.cpp revision 1.164
- Added
ConstrainFrameAndPointToAnchorSubtree() and various utility
methods.
- Modified HandleDrag() to call ConstrainFrameAndPointToAnchorSubtree().
layout/html/base/src/nsFrame.cpp revision 3.176
- Modified
GetContentAndOffsetFromPoint() to skip over generated
and anonymous
content frames.
layout/html/base/src/nsGfxScrollFrame.cpp revision 3.15
layout/html/base/src/nsGfxScrollFrame.h revision 3.8
- Override the
default nsFrame::GetContentAndOffsetsFromPoint()
implementation with a
version that calls
mInner->GetContentAndOffsetsFromPoint(). This
prevents the
scrollbar frames from being included in the search.
view/src/nsScrollPortView.cpp revision 3.6
- Fixed bug where
GetContainerSize() was returning the size of the
ScrollPortView
instead of the ScrolledView.
r=evaughan@netscape.com??????
Updated•25 years ago
|
Status: RESOLVED → VERIFIED
Comment 12•25 years ago
|
||
Verified fixed using:
- 1999112208 Linux
- 1999111909 Windows NT 4.0 SP5
- 1999112208 Mac OS
Please note that, on Linux, I was able to put Mozilla in a state in which
selection autoscrolling no longer functioned. However, I can't reproduce that
state.
(There are also known visual corruption issues during an autoscrolling drag
select on Mac OS & Linux.)
You need to log in
before you can comment on or make changes to this bug.
Description
•