From a497b033dcfe5210e36aa360a60834ad6af8e434 Mon Sep 17 00:00:00 2001 From: Cameron McCormack Date: Fri, 16 Oct 2020 12:49:47 +0000 Subject: [PATCH] style: Use separate Scrollbar and ThemedScrollbar ColorIDs. Otherwise on Windows, we have a ColorID::Scrollbar but not any of the other scrollbar part colors, and the Windows-provided value for Scrollbar doesn't work well with the default values for the other scrollbar parts that come from the non-native theme. Differential Revision: https://phabricator.services.mozilla.com/D93728 --- components/style/values/specified/color.rs | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/components/style/values/specified/color.rs b/components/style/values/specified/color.rs index d1f27eeb503..e44568fb5bf 100644 --- a/components/style/values/specified/color.rs +++ b/components/style/values/specified/color.rs @@ -108,15 +108,17 @@ pub enum SystemColor { #[css(skip)] SpellCheckerUnderline, #[css(skip)] - ScrollbarInactive, + ThemedScrollbar, #[css(skip)] - ScrollbarThumb, + ThemedScrollbarInactive, #[css(skip)] - ScrollbarThumbHover, + ThemedScrollbarThumb, #[css(skip)] - ScrollbarThumbActive, + ThemedScrollbarThumbHover, #[css(skip)] - ScrollbarThumbInactive, + ThemedScrollbarThumbActive, + #[css(skip)] + ThemedScrollbarThumbInactive, Activeborder, Activecaption, Appworkspace,