helium/ui: fix avatar button bg colors in toolbar

normal state no longer has a background and guest state is way more bearable now
This commit is contained in:
wukko
2025-07-13 23:08:38 +06:00
parent 6e8168f3bc
commit 4439e6c91b
2 changed files with 31 additions and 0 deletions

View File

@@ -0,0 +1,30 @@
--- a/chrome/browser/ui/views/profiles/avatar_toolbar_button_delegate.cc
+++ b/chrome/browser/ui/views/profiles/avatar_toolbar_button_delegate.cc
@@ -1753,7 +1753,7 @@ AvatarToolbarButtonDelegate::GetTextAndC
break;
}
case ButtonState::kNormal:
- color = color_provider->GetColor(kColorAvatarButtonHighlightNormal);
+ color = std::nullopt;
break;
#if BUILDFLAG(ENABLE_DICE_SUPPORT)
case ButtonState::kHistorySyncOptin: {
@@ -1989,7 +1989,6 @@ ui::ImageModel AvatarToolbarButtonDelega
bool AvatarToolbarButtonDelegate::ShouldPaintBorder() const {
switch (state_manager_->GetButtonActiveState()) {
case ButtonState::kGuestSession:
- return true;
case ButtonState::kShowIdentityName:
case ButtonState::kNormal:
#if BUILDFLAG(ENABLE_DICE_SUPPORT)
--- a/chrome/browser/profiles/profile_avatar_icon_util.cc
+++ b/chrome/browser/profiles/profile_avatar_icon_util.cc
@@ -436,7 +436,7 @@ ui::ImageModel GetGuestAvatar(int size)
if (base::FeatureList::IsEnabled(switches::kEnableImprovedGuestProfileMenu)) {
// Guest profiles generally use the default theme, no need to go through the
// `ThemeService`.
- color_id = ui::kColorSysPrimary;
+ color_id = ui::kColorMenuIcon;
vector_icon = &kAccountBoxIcon;
}
return ui::ImageModel::FromVectorIcon(*vector_icon, color_id, size);

View File

@@ -206,3 +206,4 @@ helium/ui/fix-omnibox-tabs-icon.patch
helium/ui/smaller-window-grab-handle.patch
helium/ui/fix-pwa-toolbar.patch
helium/ui/top-container.patch
helium/ui/toolbar-avatar-button.patch