diff --git a/components/style/properties/cascade.rs b/components/style/properties/cascade.rs index 0834da93e89..3921a4034ed 100644 --- a/components/style/properties/cascade.rs +++ b/components/style/properties/cascade.rs @@ -465,17 +465,20 @@ fn tweak_when_ignoring_colors( } }, _ => { - // We honor transparent and system colors more generally for all - // colors. + // We honor system colors more generally for all colors. // - // NOTE(emilio): This doesn't handle caret-color and - // accent-color because those use a slightly different syntax - // ( | auto for example). That's probably fine though, as - // using a system color for caret-color doesn't make sense (using - // currentColor is fine), and we ignore accent-color in - // high-contrast-mode anyways. + // We used to honor transparent but that causes accessibility + // regressions like bug 1740924. + // + // NOTE(emilio): This doesn't handle caret-color and accent-color + // because those use a slightly different syntax ( | auto for + // example). + // + // That's probably fine though, as using a system color for + // caret-color doesn't make sense (using currentColor is fine), and + // we ignore accent-color in high-contrast-mode anyways. if let Some(color) = declaration.color_value() { - if color.is_system() || alpha_channel(color, context) == 0 { + if color.is_system() { return; } }