The panel's current background works well in average situations, but in high contrast mode I noticed that certain color combinations in the web content can be pretty distracting behind the panel.

Comment on attachment 8778384 [details] [diff] [review] patch Review of attachment 8778384 [details] [diff] [review]: ----------------------------------------------------------------- I'm not really aware of the percentage-based rgb notation being commonly used, so I thought you'd take this opportunity to change it to real values. If possible, can you do that?

Certainly possible, but for made-up values rather than colors exported from a graphics editor, I prefer percentages over a 255-based scale where I need to start calculating to get my 33% gray tone for instance. We have no rule for this and use pretty much all available notations all over the place. If we were to create a rule, I'd lobby for HSL and just avoid RGB and HEX notations altogether.

Pushed by dgottwald@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/d54bb89e78af Make Ctrl+Tab panel background more opaque. r=mdeboer

Comment on attachment 8778384 [details] [diff] [review] patch Approval Request Comment [Feature/regressing bug #]: bug 1116787 [User impact if declined]: see comment 0. bug 1116787 is in the release notes, so this feature will get more attention. would be nice to have it polished. [Describe test coverage new/current, TreeHerder]: n/a [Risks and why]: trivial adjustment of a few CSS colors, basically no risk [String/UUID change made/needed]: none

Pushed by dgottwald@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/5648449ab916 followup, use hsl instead of rgb color notation

Comment on attachment 8778384 [details] [diff] [review] patch CSS only, low risk, Aurora50+