Open
Bug 426590
Opened 16 years ago
Updated 2 years ago
Unwanted scrolling when colorpicker popup is inside page that can scroll
Categories
(Core :: XUL, defect)
Tracking
()
NEW
People
(Reporter: martijn.martijn, Unassigned)
References
Details
(Keywords: testcase, Whiteboard: [missed 1.9 checkin])
Attachments
(2 files)
622 bytes,
text/html
|
Details | |
2.23 KB,
patch
|
enndeakin
:
review+
|
Details | Diff | Splinter Review |
See testcase, when pressing the down key and then the down, right, left, Pg up or Pg Dn key, the page is scrolling left, right, up or down, which you wouldn't expect when the popup is open.
Reporter | ||
Comment 1•16 years ago
|
||
Reporter | ||
Updated•16 years ago
|
Attachment #313154 -
Flags: review?(enndeakin)
Updated•16 years ago
|
Attachment #313154 -
Flags: review?(enndeakin) → review+
Reporter | ||
Updated•16 years ago
|
Attachment #313154 -
Flags: approval1.9?
Comment 2•16 years ago
|
||
While there is a testcase here that shows the problem. Can we get one in our test suite that's automated? Approval pending answer.
Reporter | ||
Comment 3•16 years ago
|
||
Sure, that is possible. As soon as I have time, I'll try to make one.
Comment 4•16 years ago
|
||
Comment on attachment 313154 [details] [diff] [review] patch a=beltzner assuming the reftest comes along with it
Attachment #313154 -
Flags: approval1.9? → approval1.9+
Updated•16 years ago
|
Assignee: nobody → martijn.martijn
Updated•16 years ago
|
Whiteboard: [missed 1.9 checkin]
Comment 5•16 years ago
|
||
Comment on attachment 313154 [details] [diff] [review] patch Removing approval since this missed the 1.9 cutoff.
Attachment #313154 -
Flags: approval1.9+
Comment 6•16 years ago
|
||
mw22, can you write the unit test and land this on Hg?
Component: XP Toolkit/Widgets: XUL → XUL
QA Contact: xptoolkit.xul → xptoolkit.widgets
Comment 8•16 years ago
|
||
When this patch lands we should remove change in the test for the colorpicker introduced in bug 455871 attachment 344994 [details] [diff] [review].
Comment 9•2 years ago
|
||
The bug assignee is inactive on Bugzilla, so the assignee is being reset.
Assignee: martijn.martijn → nobody
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•