From c9a0eaf2c03572cb3fdef267f3e371eec612f1fc Mon Sep 17 00:00:00 2001 From: jj Date: Fri, 14 Nov 2025 22:16:18 +0000 Subject: [PATCH 1/2] helium/cat: make common icons always stay unburgered --- .../experiments/compact-action-toolbar.patch | 21 +++++++++++++++++-- 1 file changed, 19 insertions(+), 2 deletions(-) diff --git a/patches/helium/ui/experiments/compact-action-toolbar.patch b/patches/helium/ui/experiments/compact-action-toolbar.patch index bac24559..f06379b8 100644 --- a/patches/helium/ui/experiments/compact-action-toolbar.patch +++ b/patches/helium/ui/experiments/compact-action-toolbar.patch @@ -163,7 +163,24 @@ show_avatar_button_.Init( prefs::kShowAvatarButton, prefs, base::BindRepeating(&ToolbarView::OnShowAvatarButtonChanged, -@@ -700,6 +723,24 @@ bool ToolbarView::IsRectInWindowCaption( +@@ -564,10 +587,16 @@ void ToolbarView::Init() { + + InitLayout(); + ++ ++ auto flex_preferred = views::FlexSpecification( ++ views::MinimumFlexSizeRule::kPreferred, ++ views::MaximumFlexSizeRule::kPreferred); ++ + for (auto* button : std::array{back_, forward_, reload_, + home_, avatar_}) { + if (button) { + button->set_tag(GetViewCommandMap().at(button->GetID())); ++ button->SetProperty(views::kFlexBehaviorKey, flex_preferred); + } + } + if (browser_view_->GetSupportsTabStrip()) { +@@ -700,6 +729,24 @@ bool ToolbarView::IsRectInWindowCaption( return gfx::ToEnclosingRect(rect_in_target_coords_f); }; @@ -188,7 +205,7 @@ // Check each child view in container_view_ to see if the rect intersects with // any clickable elements. If it does, check if the click is actually on that // element. False if on a clickable element, true if not on a clickable element. -@@ -998,6 +1039,12 @@ void ToolbarView::InitLayout() { +@@ -998,6 +1045,12 @@ void ToolbarView::InitLayout() { location_bar_->SetProperty(views::kMarginsKey, gfx::Insets::VH(0, location_bar_margin)); From 0008cd2ceceb2a1cf6dec6efa8039cd32c3fe84a Mon Sep 17 00:00:00 2001 From: wukko Date: Tue, 18 Nov 2025 16:37:38 +0600 Subject: [PATCH 2/2] helium/cat: remove extra empty line --- patches/helium/ui/experiments/compact-action-toolbar.patch | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/patches/helium/ui/experiments/compact-action-toolbar.patch b/patches/helium/ui/experiments/compact-action-toolbar.patch index 5f0b03f8..a1e98ffc 100644 --- a/patches/helium/ui/experiments/compact-action-toolbar.patch +++ b/patches/helium/ui/experiments/compact-action-toolbar.patch @@ -172,11 +172,10 @@ show_avatar_button_.Init( prefs::kShowAvatarButton, prefs, base::BindRepeating(&ToolbarView::OnShowAvatarButtonChanged, -@@ -564,10 +587,16 @@ void ToolbarView::Init() { +@@ -564,10 +587,15 @@ void ToolbarView::Init() { InitLayout(); -+ + auto flex_preferred = views::FlexSpecification( + views::MinimumFlexSizeRule::kPreferred, + views::MaximumFlexSizeRule::kPreferred); @@ -189,7 +188,7 @@ } } if (browser_view_->GetSupportsTabStrip()) { -@@ -700,6 +729,24 @@ bool ToolbarView::IsRectInWindowCaption( +@@ -700,6 +728,24 @@ bool ToolbarView::IsRectInWindowCaption( return gfx::ToEnclosingRect(rect_in_target_coords_f); }; @@ -214,7 +213,7 @@ // Check each child view in container_view_ to see if the rect intersects with // any clickable elements. If it does, check if the click is actually on that // element. False if on a clickable element, true if not on a clickable element. -@@ -998,6 +1045,12 @@ void ToolbarView::InitLayout() { +@@ -998,6 +1044,12 @@ void ToolbarView::InitLayout() { location_bar_->SetProperty(views::kMarginsKey, gfx::Insets::VH(0, location_bar_margin));