Open Bug 675456 Opened 13 years ago Updated 2 years ago

No scrollbar movement when dragging selection below current view

Categories

(Core :: DOM: Selection, defect)

x86
Windows 2000
defect

Tracking

()

People

(Reporter: JoeS1, Unassigned)

References

()

Details

(Keywords: regression)

Attachments

(2 files)

Attached file long text for demo
If the compose window has content long enough to cause a scrollbar, attempting to select content by left-click drag will not move the scrollbar.

This is seen in seamonkey composer as well as current trun thunderbird compose.

regression range:
works Mozilla/5.0 (Windows NT 5.0; rv:8.0a1) Gecko/20110706 Thunderbird/8.0a1 ID:20110706030443
fails Mozilla/5.0 (Windows NT 5.0; rv:8.0a1) Gecko/20110707 Thunderbird/8.0a1 ID:20110707030312

Reply to the testcase attachment for demo.
Assignee: nobody → masayuki
Component: Message Compose Window → Selection
Product: Thunderbird → Core
QA Contact: message-compose → selection
Target Milestone: Thunderbird 8.0 → ---
I think that I failed to get selection root element for this case.
Status: NEW → ASSIGNED
Attached patch PatchSplinter Review
The cause is that root frame is scrollable but body is the selection root.

When capturing content is body element of the document, FindNearestScrollableFrameForSelection() should return the document's root scrollable frame if it doesn't fine any scrollable frame between the frame for body and aFrame. This doesn't change selection root. I.e., this patch doesn't change the selectable area from current trunk build. So, this should be safer than change it.

# I confirmed that this bug fixes the original reported case (i.e., compose window of tb) actually.
Attachment #549859 - Flags: review?(Olli.Pettay)
Comment on attachment 549859 [details] [diff] [review]
Patch

Hum, I thought I reviewed this already last week :/
Attachment #549859 - Flags: review?(Olli.Pettay) → review+

Resetting assignee which I don't work on in this several months.

Assignee: masayuki → nobody
Status: ASSIGNED → NEW
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: