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)
Tracking
()
NEW
People
(Reporter: JoeS1, Unassigned)
References
()
Details
(Keywords: regression)
Attachments
(2 files)
4.90 KB,
message/rfc822
|
Details | |
17.71 KB,
patch
|
smaug
:
review+
|
Details | Diff | Splinter Review |
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.
Updated•13 years ago
|
Assignee: nobody → masayuki
Component: Message Compose Window → Selection
Product: Thunderbird → Core
QA Contact: message-compose → selection
Target Milestone: Thunderbird 8.0 → ---
Comment 1•13 years ago
|
||
I think that I failed to get selection root element for this case.
Status: NEW → ASSIGNED
Updated•13 years ago
|
Comment 2•13 years ago
|
||
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 3•13 years ago
|
||
Comment on attachment 549859 [details] [diff] [review] Patch Hum, I thought I reviewed this already last week :/
Attachment #549859 -
Flags: review?(Olli.Pettay) → review+
Comment 4•4 years ago
|
||
Resetting assignee which I don't work on in this several months.
Assignee: masayuki → nobody
Status: ASSIGNED → NEW
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•