There is no class usable to display a <menulist/> on a dark background.
Created attachment 405430 [details] [diff] [review] fix The patch adds a "menu-dark" class to style menulists on dark backgrounds
Comment on attachment 405430 [details] [diff] [review] fix I think we should name the new class "button-dark" for consistency. We can change it to "control-dark" or something if we want it to be generic. r+ with the name change, which I can do on checkin
Attachment #405430 - Flags: review+
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → B5
Would about:config on an inverted theme be a proper testcase for this patch?
(In reply to comment #4) > Would about:config on an inverted theme be a proper testcase for this patch? Nope. There are no menulists in the about:config
bug 517459 has code and a screenshot of dark menulists working
You need to log in before you can comment on or make changes to this bug.