Created attachment 353052 [details] [diff] [review] patch v1 Since bug 391984 popup menu transparency is no longer implemented using -[NSWindow setAlphaValue:]; instead, we now make the window translucent and let -moz-appearance: menupopup / menuitem manage the transparency. However, since the menuitems draw on top of the menupopup, their opacity gets multiplied. That means that the popup is now more transparent under the top and bottom padding than under the menuitems. This patch adds a CGContextClearRect when drawing menuitems, as suggested in bug 391984 comment 5. This avoids the multiplying opacity problem. Furthermore, I'm removing the popup window alpha stuff because we don't need it any more.
Attachment #353052 - Flags: superreview?(roc) → superreview+
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.2a1
Attachment #353052 - Flags: approval1.9.1?
Comment on attachment 353052 [details] [diff] [review] patch v1 a191=beltzner
Attachment #353052 - Flags: approval1.9.1? → approval1.9.1+
Landed on 1.9.1: http://hg.mozilla.org/releases/mozilla-1.9.1/rev/185e694dc180
You need to log in before you can comment on or make changes to this bug.