Closed
Bug 829886
Opened 12 years ago
Closed 12 years ago
Can't select options in a select list whose parent is translated with a transform on mac
Categories
(Core :: Layout: Form Controls, defect)
Tracking
()
VERIFIED
FIXED
mozilla21
People
(Reporter: tnikkel, Assigned: tnikkel)
References
Details
Attachments
(1 file)
2.23 KB,
patch
|
smichaud
:
review+
lsblakk
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
This is bug 759993 but for mac only.
Assignee | ||
Comment 1•12 years ago
|
||
The problem is that nsChildView always seems to return 0,0 for the position from GetScreenBounds and GetClientBounds.
Assignee | ||
Comment 2•12 years ago
|
||
Assignee: nobody → tnikkel
Attachment #702442 -
Flags: review?(smichaud)
Comment 3•12 years ago
|
||
Comment on attachment 702442 [details] [diff] [review]
patch
Looks fine to me.
Attachment #702442 -
Flags: review?(smichaud) → review+
Assignee | ||
Comment 4•12 years ago
|
||
Comment 5•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
Assignee | ||
Comment 6•12 years ago
|
||
Comment on attachment 702442 [details] [diff] [review]
patch
This turns out to fix the regression bug 843675, so because of that I think we should consider this for uplift to beta (it is on aurora already since it landed before the migration).
[Approval Request Comment]
Bug caused by (feature/regressing bug #): bug 843675 (the bug we aim to fix) was caused by bug 759993
User impact if declined: select drop downs that have CSS transform scale(1) won't work on Mac, which seems minor but the issue was reported a few days after it appeared on the release channel, so it appears this is an issue for some people.
Testing completed (on m-c, etc.): we've had this on central for over a month now and no problems reported
Risk to taking this patch (and alternatives if risky): should be safe, only risk is if we had code depending on the old broken behaviour. we haven't found any in a month, so probably not.
String or UUID changes made by this patch: none
Attachment #702442 -
Flags: approval-mozilla-beta?
Updated•12 years ago
|
status-firefox19:
--- → wontfix
status-firefox20:
--- → affected
status-firefox21:
--- → fixed
tracking-firefox20:
--- → +
tracking-firefox21:
--- → +
Comment 7•12 years ago
|
||
Comment on attachment 702442 [details] [diff] [review]
patch
Yes, let's get this 19 regression fix into FF20.
Attachment #702442 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Assignee | ||
Comment 8•12 years ago
|
||
Comment 9•12 years ago
|
||
Verified fixed FF 20b2, 21.0a2 (2013-02-27) Mac OS X 10.8.2
You need to log in
before you can comment on or make changes to this bug.
Description
•