Skip to content

Commit

Permalink
Merge pull request #24086 from brave/update_app_menu_update_alert_color
Browse files Browse the repository at this point in the history
Update update alert color from app menu for light theme
  • Loading branch information
simonhong authored Jun 12, 2024
2 parents 706f518 + 6a99804 commit ced879c
Showing 1 changed file with 9 additions and 9 deletions.
18 changes: 9 additions & 9 deletions browser/ui/color/brave_color_mixer.cc
Original file line number Diff line number Diff line change
Expand Up @@ -448,17 +448,17 @@ void AddBravifiedChromeThemeColorMixer(ui::ColorProvider* provider,
AddMaterialSidePanelColorMixer(provider, key);
}

// Upstream started to use same colors for all severity. But, we want to use
// upstream's previous non-material color to app menu highlight. These colors
// are copied from AddChromeColorMixer() because it's overwritten by
// AddMaterialChromeColorMixer().
// TODO(simonhong): Use leo color when it's ready.
ui::ColorMixer& mixer = provider->AddMixer();
mixer[kColorAppMenuHighlightSeverityLow] = AdjustHighlightColorForContrast(
ui::kColorAlertLowSeverity, kColorToolbar);
mixer[kColorAppMenuHighlightSeverityLow] = {
PickColorContrastingToToolbar(key, mixer, SkColorSetRGB(0x00, 0x46, 0x07),
SkColorSetRGB(0x58, 0xE1, 0x55))};
mixer[kColorAppMenuHighlightSeverityMedium] = {
PickColorContrastingToToolbar(key, mixer, SkColorSetRGB(0x4A, 0x39, 0x00),
SkColorSetRGB(0xF1, 0xC0, 0x0F))};
mixer[kColorAppMenuHighlightSeverityHigh] = {
kColorAvatarButtonHighlightSyncError};
mixer[kColorAppMenuHighlightSeverityMedium] = AdjustHighlightColorForContrast(
ui::kColorAlertMediumSeverityIcon, kColorToolbar);
PickColorContrastingToToolbar(key, mixer, SkColorSetRGB(0x7D, 0x00, 0x1A),
SkColorSetRGB(0xFF, 0xB3, 0xB2))};

if (key.custom_theme) {
return;
Expand Down

0 comments on commit ced879c

Please sign in to comment.