Commit 5e03844d authored by Yulun Wu's avatar Yulun Wu Committed by Commit Bot

Update login shelf icons and coloring for light/dark mode.

Bug: 1143063
Change-Id: Icd0ec2088508d633757ed20749790b80a1dc09d3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2508558Reviewed-by: default avatarMin Chen <minch@chromium.org>
Reviewed-by: default avatarRoman Sorokin [CET] <rsorokin@chromium.org>
Reviewed-by: default avatarThomas Tellier <tellier@google.com>
Reviewed-by: default avatarXiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Yulun Wu <yulunwu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#825547}
parent d1b5a6ec
...@@ -120,9 +120,8 @@ void LoginBigUserView::OnWallpaperBlurChanged() { ...@@ -120,9 +120,8 @@ void LoginBigUserView::OnWallpaperBlurChanged() {
layer()->SetFillsBoundsOpaquely(false); layer()->SetFillsBoundsOpaquely(false);
SetBackground(views::CreateBackgroundFromPainter( SetBackground(views::CreateBackgroundFromPainter(
views::Painter::CreateSolidRoundRectPainter( views::Painter::CreateSolidRoundRectPainter(
SkColorSetA(DeprecatedGetLoginBackgroundBaseColor( AshColorProvider::Get()->GetShieldLayerColor(
kLoginBackgroundBaseColor), AshColorProvider::ShieldLayerType::kShield80),
login_constants::kNonBlurredWallpaperBackgroundAlpha),
login_constants::kNonBlurredWallpaperBackgroundRadiusDp))); login_constants::kNonBlurredWallpaperBackgroundRadiusDp)));
} }
} }
......
...@@ -254,12 +254,9 @@ ScrollableUsersListView::GradientParams::BuildForStyle( ...@@ -254,12 +254,9 @@ ScrollableUsersListView::GradientParams::BuildForStyle(
color_utils::ColorProfile(color_utils::LumaRange::DARK, color_utils::ColorProfile(color_utils::LumaRange::DARK,
color_utils::SaturationRange::MUTED)); color_utils::SaturationRange::MUTED));
SkColor tint_color = color_utils::GetResultingPaintColor( SkColor tint_color = color_utils::GetResultingPaintColor(
SkColorSetA( AshColorProvider::Get()->GetShieldLayerColor(
DeprecatedGetLoginBackgroundBaseColor(kLoginBackgroundBaseColor), AshColorProvider::ShieldLayerType::kShield80),
login_constants::kTranslucentColorDarkenAlpha),
SkColorSetA(dark_muted_color, SK_AlphaOPAQUE)); SkColorSetA(dark_muted_color, SK_AlphaOPAQUE));
tint_color =
SkColorSetA(tint_color, login_constants::kScrollTranslucentAlpha);
GradientParams params; GradientParams params;
params.color_from = dark_muted_color; params.color_from = dark_muted_color;
...@@ -427,9 +424,8 @@ void ScrollableUsersListView::OnPaintBackground(gfx::Canvas* canvas) { ...@@ -427,9 +424,8 @@ void ScrollableUsersListView::OnPaintBackground(gfx::Canvas* canvas) {
cc::PaintFlags flags; cc::PaintFlags flags;
flags.setAntiAlias(true); flags.setAntiAlias(true);
flags.setStyle(cc::PaintFlags::kFill_Style); flags.setStyle(cc::PaintFlags::kFill_Style);
flags.setColor(SkColorSetA( flags.setColor(AshColorProvider::Get()->GetShieldLayerColor(
DeprecatedGetLoginBackgroundBaseColor(kLoginBackgroundBaseColor), AshColorProvider::ShieldLayerType::kShield80));
login_constants::kNonBlurredWallpaperBackgroundAlpha));
canvas->DrawRoundRect( canvas->DrawRoundRect(
render_bounds, login_constants::kNonBlurredWallpaperBackgroundRadiusDp, render_bounds, login_constants::kNonBlurredWallpaperBackgroundRadiusDp,
flags); flags);
......
...@@ -94,7 +94,8 @@ SkColor GetButtonBackgroundColor() { ...@@ -94,7 +94,8 @@ SkColor GetButtonBackgroundColor() {
session_manager::SessionState::OOBE) { session_manager::SessionState::OOBE) {
return SkColorSetA(SK_ColorBLACK, 16); // 6% opacity return SkColorSetA(SK_ColorBLACK, 16); // 6% opacity
} }
return DeprecatedGetLoginBackgroundBaseColor(kLoginButtonBackgroundBaseColor); return AshColorProvider::Get()->GetControlsLayerColor(
AshColorProvider::ControlsLayerType::kControlBackgroundColorInactive);
} }
LoginMetricsRecorder::ShelfButtonClickTarget GetUserClickTarget(int button_id) { LoginMetricsRecorder::ShelfButtonClickTarget GetUserClickTarget(int button_id) {
...@@ -180,13 +181,8 @@ class LoginShelfButton : public views::LabelButton { ...@@ -180,13 +181,8 @@ class LoginShelfButton : public views::LabelButton {
text_resource_id_(text_resource_id), text_resource_id_(text_resource_id),
icon_(icon) { icon_(icon) {
SetAccessibleName(GetText()); SetAccessibleName(GetText());
SkColor button_icon_color = GetButtonIconColor(); AshColorProvider::Get()->DecoratePillButton(this, &icon);
SetImage(views::Button::STATE_NORMAL,
gfx::CreateVectorIcon(icon, button_icon_color));
SetImage(views::Button::STATE_DISABLED,
gfx::CreateVectorIcon(
icon, SkColorSetA(button_icon_color,
login_constants::kButtonDisabledAlpha)));
SetFocusBehavior(FocusBehavior::ALWAYS); SetFocusBehavior(FocusBehavior::ALWAYS);
SetInstallFocusRingOnFocus(true); SetInstallFocusRingOnFocus(true);
views::InstallRoundRectHighlightPathGenerator( views::InstallRoundRectHighlightPathGenerator(
...@@ -209,11 +205,6 @@ class LoginShelfButton : public views::LabelButton { ...@@ -209,11 +205,6 @@ class LoginShelfButton : public views::LabelButton {
SetImageLabelSpacing(kImageLabelSpacingDp); SetImageLabelSpacing(kImageLabelSpacingDp);
SkColor button_text_color = GetButtonTextColor();
SetEnabledTextColors(button_text_color);
SetTextColor(
views::Button::STATE_DISABLED,
SkColorSetA(button_text_color, login_constants::kButtonDisabledAlpha));
label()->SetFontList(views::Label::GetDefaultFontList().Derive( label()->SetFontList(views::Label::GetDefaultFontList().Derive(
1, gfx::Font::FontStyle::NORMAL, gfx::Font::Weight::NORMAL)); 1, gfx::Font::FontStyle::NORMAL, gfx::Font::Weight::NORMAL));
} }
......
...@@ -241,9 +241,8 @@ SkColor ShelfBackgroundAnimator::GetBackgroundColor( ...@@ -241,9 +241,8 @@ SkColor ShelfBackgroundAnimator::GetBackgroundColor(
shelf_target_color = SK_ColorTRANSPARENT; shelf_target_color = SK_ColorTRANSPARENT;
break; break;
case ShelfBackgroundType::kLoginNonBlurredWallpaper: case ShelfBackgroundType::kLoginNonBlurredWallpaper:
shelf_target_color = SkColorSetA( shelf_target_color = AshColorProvider::Get()->GetShieldLayerColor(
DeprecatedGetLoginBackgroundBaseColor(kLoginBackgroundBaseColor), AshColorProvider::ShieldLayerType::kShield80);
login_constants::kNonBlurredWallpaperBackgroundAlpha);
break; break;
} }
return shelf_target_color; return shelf_target_color;
......
...@@ -9,7 +9,6 @@ ...@@ -9,7 +9,6 @@
#include "ash/public/cpp/ash_constants.h" #include "ash/public/cpp/ash_constants.h"
#include "ash/public/cpp/ash_features.h" #include "ash/public/cpp/ash_features.h"
#include "ash/public/cpp/ash_pref_names.h" #include "ash/public/cpp/ash_pref_names.h"
#include "ash/public/cpp/login_constants.h"
#include "ash/session/session_controller_impl.h" #include "ash/session/session_controller_impl.h"
#include "ash/shell.h" #include "ash/shell.h"
#include "ash/system/dark_mode/color_mode_observer.h" #include "ash/system/dark_mode/color_mode_observer.h"
...@@ -391,11 +390,6 @@ void AshColorProvider::UpdateColorModeThemed(bool is_themed) { ...@@ -391,11 +390,6 @@ void AshColorProvider::UpdateColorModeThemed(bool is_themed) {
AttemptRestartChrome(); AttemptRestartChrome();
} }
SkColor AshColorProvider::GetLoginBackgroundBaseColor() const {
return IsDarkModeEnabled() ? login_constants::kDefaultBaseColor
: login_constants::kLightModeBaseColor;
}
SkColor AshColorProvider::GetBackgroundDefaultColor() const { SkColor AshColorProvider::GetBackgroundDefaultColor() const {
return IsDarkModeEnabled() ? kBackgroundColorDefaultDark return IsDarkModeEnabled() ? kBackgroundColorDefaultDark
: kBackgroundColorDefaultLight; : kBackgroundColorDefaultLight;
......
...@@ -210,9 +210,6 @@ class ASH_EXPORT AshColorProvider : public SessionObserver { ...@@ -210,9 +210,6 @@ class ASH_EXPORT AshColorProvider : public SessionObserver {
// Updates pref |kColorModeThemed| to |is_themed|. // Updates pref |kColorModeThemed| to |is_themed|.
void UpdateColorModeThemed(bool is_themed); void UpdateColorModeThemed(bool is_themed);
// Gets the background base color for login screen.
SkColor GetLoginBackgroundBaseColor() const;
private: private:
friend class ScopedLightModeAsDefault; friend class ScopedLightModeAsDefault;
......
...@@ -48,12 +48,6 @@ SkColor DeprecatedGetContentLayerColor(AshColorProvider::ContentLayerType type, ...@@ -48,12 +48,6 @@ SkColor DeprecatedGetContentLayerColor(AshColorProvider::ContentLayerType type,
return AshColorProvider::Get()->GetContentLayerColor(type); return AshColorProvider::Get()->GetContentLayerColor(type);
} }
SkColor DeprecatedGetLoginBackgroundBaseColor(SkColor default_color) {
if (!features::IsDarkLightModeEnabled())
return default_color;
return AshColorProvider::Get()->GetLoginBackgroundBaseColor();
}
SkColor DeprecatedGetInkDropBaseColor(SkColor default_color) { SkColor DeprecatedGetInkDropBaseColor(SkColor default_color) {
if (!features::IsDarkLightModeEnabled()) if (!features::IsDarkLightModeEnabled())
return default_color; return default_color;
......
...@@ -25,7 +25,6 @@ SkColor DeprecatedGetControlsLayerColor( ...@@ -25,7 +25,6 @@ SkColor DeprecatedGetControlsLayerColor(
SkColor default_color); SkColor default_color);
SkColor DeprecatedGetContentLayerColor(AshColorProvider::ContentLayerType type, SkColor DeprecatedGetContentLayerColor(AshColorProvider::ContentLayerType type,
SkColor default_color); SkColor default_color);
SkColor DeprecatedGetLoginBackgroundBaseColor(SkColor default_color);
SkColor DeprecatedGetInkDropBaseColor(SkColor default_color); SkColor DeprecatedGetInkDropBaseColor(SkColor default_color);
SkColor DeprecatedGetInkDropRippleColor(SkColor default_color); SkColor DeprecatedGetInkDropRippleColor(SkColor default_color);
SkColor DeprecatedGetInkDropHighlightColor(SkColor default_color); SkColor DeprecatedGetInkDropHighlightColor(SkColor default_color);
......
...@@ -22,38 +22,11 @@ namespace ash { ...@@ -22,38 +22,11 @@ namespace ash {
namespace { namespace {
// The value used for alpha to apply a color filter to the wallpaper in tablet
// mode. A higher number up to 255 results the color filter being more
// prominent.
constexpr int kTabletModeWallpaperAlpha = 102;
// Gets the color filter based on the state. This is used for the login, lock,
// overview and tablet mode.
SkColor GetWallpaperFilterColor() { SkColor GetWallpaperFilterColor() {
if (Shell::Get()->overview_controller()->InOverviewSession()) { return AshColorProvider::Get()->GetShieldLayerColor(
return AshColorProvider::Get()->GetShieldLayerColor( Shell::Get()->overview_controller()->InOverviewSession()
AshColorProvider::ShieldLayerType::kShield40); ? AshColorProvider::ShieldLayerType::kShield40
} : AshColorProvider::ShieldLayerType::kShield80);
SkColor darken_color =
Shell::Get()->wallpaper_controller()->GetProminentColor(
color_utils::ColorProfile(color_utils::LumaRange::DARK,
color_utils::SaturationRange::MUTED));
if (darken_color == kInvalidWallpaperColor) {
darken_color =
DeprecatedGetLoginBackgroundBaseColor(kLoginBackgroundBaseColor);
}
darken_color = color_utils::GetResultingPaintColor(
SkColorSetA(
DeprecatedGetLoginBackgroundBaseColor(kLoginBackgroundBaseColor),
login_constants::kTranslucentColorDarkenAlpha),
SkColorSetA(darken_color, 0xFF));
const int alpha = Shell::Get()->tablet_mode_controller()->InTabletMode()
? kTabletModeWallpaperAlpha
: login_constants::kTranslucentAlpha;
return SkColorSetA(darken_color, alpha);
} }
} // namespace } // namespace
......
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