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)

x86
Windows XP
defect

Tracking

()

People

(Reporter: martijn.martijn, Unassigned)

References

Details

(Keywords: testcase, Whiteboard: [missed 1.9 checkin])

Attachments

(2 files)

Attached file testcase
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.
Attached patch patchSplinter Review
Attachment #313154 - Flags: review?(enndeakin)
Attachment #313154 - Flags: review?(enndeakin) → review+
Attachment #313154 - Flags: approval1.9?
While there is a testcase here that shows the problem.  Can we get one in our test suite that's automated?  Approval pending answer.
Sure, that is possible. As soon as I have time, I'll try to make one.
Comment on attachment 313154 [details] [diff] [review]
patch

a=beltzner assuming the reftest comes along with it
Attachment #313154 - Flags: approval1.9? → approval1.9+
Assignee: nobody → martijn.martijn
Whiteboard: [missed 1.9 checkin]
Comment on attachment 313154 [details] [diff] [review]
patch

Removing approval since this missed the 1.9 cutoff.
Attachment #313154 - Flags: approval1.9+
mw22, can you write the unit test and land this on Hg?
Component: XP Toolkit/Widgets: XUL → XUL
QA Contact: xptoolkit.xul → xptoolkit.widgets
When this patch lands we should remove change in the test for the colorpicker introduced in bug 455871 attachment 344994 [details] [diff] [review].

The bug assignee is inactive on Bugzilla, so the assignee is being reset.

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

Attachment

General

Created:
Updated:
Size: