diff --git a/browser/ui/views/sidebar/sidebar_item_add_button.cc b/browser/ui/views/sidebar/sidebar_item_add_button.cc index dd7a4f52e99a..92d992bc97f3 100644 --- a/browser/ui/views/sidebar/sidebar_item_add_button.cc +++ b/browser/ui/views/sidebar/sidebar_item_add_button.cc @@ -47,6 +47,12 @@ void SidebarItemAddButton::OnGestureEvent(ui::GestureEvent* event) { } } +void SidebarItemAddButton::OnThemeChanged() { + View::OnThemeChanged(); + + UpdateButtonImages(); +} + void SidebarItemAddButton::AddedToWidget() { UpdateButtonImages(); } diff --git a/browser/ui/views/sidebar/sidebar_item_add_button.h b/browser/ui/views/sidebar/sidebar_item_add_button.h index f6899f5d5aea..9a289c31312e 100644 --- a/browser/ui/views/sidebar/sidebar_item_add_button.h +++ b/browser/ui/views/sidebar/sidebar_item_add_button.h @@ -32,6 +32,7 @@ class SidebarItemAddButton : public SidebarButtonView, void OnMouseExited(const ui::MouseEvent& event) override; void OnGestureEvent(ui::GestureEvent* event) override; void AddedToWidget() override; + void OnThemeChanged() override; // views::WidgetObserver overrides: void OnWidgetDestroying(views::Widget* widget) override; diff --git a/browser/ui/views/sidebar/sidebar_items_contents_view.cc b/browser/ui/views/sidebar/sidebar_items_contents_view.cc index ed9eefb91531..91eef0fc4b89 100644 --- a/browser/ui/views/sidebar/sidebar_items_contents_view.cc +++ b/browser/ui/views/sidebar/sidebar_items_contents_view.cc @@ -109,11 +109,6 @@ void SidebarItemsContentsView::UpdateAllBuiltInItemsViewState() { if (!sidebar::IsBuiltInType(item)) continue; - if (item.open_in_panel) { - UpdateItemViewStateAt(item_index, item_index == active_index); - continue; - } - // If browser window has tab that loads brave talk, brave talk panel icon // will use colored one for normal state also. if (item.built_in_item_type == @@ -122,7 +117,10 @@ void SidebarItemsContentsView::UpdateAllBuiltInItemsViewState() { item_index, browser_->sidebar_controller()->DoesBrowserHaveOpenedTabForItem( item)); + continue; } + + UpdateItemViewStateAt(item_index, item_index == active_index); } }