Commit 232549f0 authored by Mathieu Perreault's avatar Mathieu Perreault Committed by Commit Bot

[Theme] Adjust default colors (green and red) to be lighter

According to designers and internal spec (https://goo.gl/oRp6PA), some
colors are a shade lighter.

Bug: 850571
Test: Visual
Change-Id: I2c593ed0019c9497873f0f8efa42b592e187d89b
Reviewed-on: https://chromium-review.googlesource.com/1099687
Commit-Queue: Peter Kasting <pkasting@chromium.org>
Reviewed-by: default avatarPeter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#568306}
parent 04b5203d
...@@ -74,9 +74,9 @@ SkColor GetHarmonyTextColorForNonStandardNativeTheme( ...@@ -74,9 +74,9 @@ SkColor GetHarmonyTextColorForNonStandardNativeTheme(
color_id = ui::NativeTheme::kColorId_LinkEnabled; color_id = ui::NativeTheme::kColorId_LinkEnabled;
break; break;
case STYLE_RED: case STYLE_RED:
return inverted_scheme ? gfx::kGoogleRed300 : gfx::kGoogleRed700; return inverted_scheme ? gfx::kGoogleRed300 : gfx::kGoogleRed600;
case STYLE_GREEN: case STYLE_GREEN:
return inverted_scheme ? gfx::kGoogleGreen300 : gfx::kGoogleGreen700; return inverted_scheme ? gfx::kGoogleGreen300 : gfx::kGoogleGreen600;
} }
return theme.GetSystemColor(color_id); return theme.GetSystemColor(color_id);
} }
......
...@@ -352,11 +352,11 @@ SkColor GetAuraColor(NativeTheme::ColorId color_id, ...@@ -352,11 +352,11 @@ SkColor GetAuraColor(NativeTheme::ColorId color_id,
// Alert icon colors // Alert icon colors
case NativeTheme::kColorId_AlertSeverityLow: case NativeTheme::kColorId_AlertSeverityLow:
return gfx::kGoogleGreen700; return gfx::kGoogleGreen600;
case NativeTheme::kColorId_AlertSeverityMedium: case NativeTheme::kColorId_AlertSeverityMedium:
return gfx::kGoogleYellow700; return gfx::kGoogleYellow700;
case NativeTheme::kColorId_AlertSeverityHigh: case NativeTheme::kColorId_AlertSeverityHigh:
return gfx::kGoogleRed700; return gfx::kGoogleRed600;
case NativeTheme::kColorId_NumColors: case NativeTheme::kColorId_NumColors:
break; break;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment