[DOGFOOD] Can't use text selection to scroll in textarea

VERIFIED FIXED in M12

Status

()

Core
Selection
P1
critical
VERIFIED FIXED
19 years ago
19 years ago

People

(Reporter: cpratt, Assigned: kinmoz)

Tracking

Trunk
All
Mac System 8.5
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

19 years ago
build id: 1999042608
reproduces on mac only. works as expected on windows and linux.

to reproduce this, load the url above (or use any page that contains a textarea
with more text in it than can be displayed on screen). click and drag inside the
textarea, moving the mouse down to scroll down and read all of the text in the
textarea. result: you can't - it just highlights down to the last visible line
in the textarea.

expected result: as you continue to drag the mouse down, the text in the
textarea should highlight and scroll down until you reach the end of the text.

Comment 1

19 years ago
Drag-selection also doesn't work for web pages, either.

The funny part is that this is the bug example used for the bug writing
guidelines themselves, but I can't find a single actual bug documenting this
issue. ;) I'll go write one.

Comment 2

19 years ago
[actually, Simon says that the drag-selection not working on web pages --- as
opposed to textareas --- is a known issue and will be implemented in Msometime,
so I won't write up a bug afterall. ;-]

Updated

19 years ago
Status: NEW → ASSIGNED

Comment 3

19 years ago
yes someday .. someeday...

Updated

19 years ago
Summary: [PP] mac only: can't use text selection to scroll in textarea → Not just Mac: can't use text selection to scroll in textarea
Target Milestone: M7

Comment 4

19 years ago
this will not be mac specific and I want to use this to track webshells
reluctance to scroll with selection

Updated

19 years ago
Hardware: Macintosh → All

Comment 5

19 years ago
so when you say webshell you mean that drag-selecting on a web won't cause the window to scroll? Do we need a seperate bug
then for the text-area selection not initiating a scroll or will we get that for free when webshell starts behaving?

Changed platform to ALL

Updated

19 years ago
Summary: Not just Mac: can't use text selection to scroll in textarea → Can't use text selection to scroll in textarea

Updated

19 years ago
Target Milestone: M7 → M9

Comment 6

19 years ago
this will require extensive scroll work. i think this will not happen until troy
gets back in about 2 weeks so i am moving this to m9

Comment 7

19 years ago
Per a request from Selection and Search component eng (mjudge) and qa (elig),
moving all "Selection and Search" bugs to new "Selection" component.  Original
"Selection and Search" component will be retired.

Updated

19 years ago
QA Contact: claudius → elig

Comment 8

19 years ago
[QA Assigning to self.]

Comment 9

19 years ago
Kin, is this part of the scrollFrameToView work you are doing?
(Assignee)

Comment 10

19 years ago
I am currently working on a couple of methods ScrollRangeIntoView() and
ScrollRectIntoView() that can be used to fix these problems.

After I get these done, I'll work on handling the events that cause
auto-scrolling.

Updated

19 years ago
Target Milestone: M9 → M10

Updated

19 years ago
Severity: normal → critical
Priority: P3 → P1
Summary: Can't use text selection to scroll in textarea → [DOGFOOD] Can't use text selection to scroll in textarea
Target Milestone: M10 → M11

Updated

19 years ago
Assignee: mjudge → kin
Status: ASSIGNED → NEW

Comment 11

19 years ago
kin has solution for this he will check in in m12
(Assignee)

Updated

19 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 12

19 years ago
Accepting bug. Cc mjudge@netscape.com.
(Assignee)

Updated

19 years ago
Target Milestone: M11 → M12
(Assignee)

Comment 13

19 years ago
Setting milestone to M12.
(Assignee)

Updated

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

Comment 14

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

Updated

19 years ago
Status: RESOLVED → VERIFIED

Comment 15

19 years ago
Verified on:
	1999100708 Mac OS 8.6 & Linux (RH 6.0/GNOME)
	1999100708 Windows NT 4.0 SP5

Note that the drag-selection highlighting issue from bug #15384 occurs when
reproducing this bug. To be sure, I've left a note in that bug report to double-
check this one when it gets verified.

Comment 16

19 years ago
Test case wasn't preserved by bug reporter, so the side-issue can't be verified
as part of verifying bug #15384.
(Reporter)

Comment 17

19 years ago
Created attachment 5859 [details]
original test case

Comment 18

19 years ago
Thanks. Re-verified fixed on this morning's Mac OS & Linux builds.

Updated

18 years ago
Blocks: 36868
You need to log in before you can comment on or make changes to this bug.