From 2da77405adec4cbbc634518fbd5b149b95f31c05 Mon Sep 17 00:00:00 2001
From: Pedro Pinto Silva <pedro.silva@collabora.com>
Date: Tue, 28 May 2024 17:09:49 +0200
Subject: [PATCH] Formula bar: buttons: make grid on equably sized columns and
 fix btn

- Toolbox: Remove height min-content as it's prone to problems and different
outcomes depending on the context. Plus we don't need it if we enforce
it via grid
- Center all elements
  - Including unotoolbutton

Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: Iadc55f0d253dd4a06c11ad8cd062c02863b1d12f
---
 browser/css/jsdialogs.css   | 15 +++++++++++++--
 browser/css/spreadsheet.css |  2 --
 2 files changed, 13 insertions(+), 4 deletions(-)

diff --git a/browser/css/jsdialogs.css b/browser/css/jsdialogs.css
index 56e74a1ad..6893cdb8d 100644
--- a/browser/css/jsdialogs.css
+++ b/browser/css/jsdialogs.css
@@ -1846,10 +1846,21 @@ input[type='number']:hover::-webkit-outer-spin-button {
 	border-radius: var(--border-radius) 0 var(--border-radius) var(--border-radius) !important;
 }
 
-#formulabar-buttons-toolbox.formulabar {
+#formulabar-buttons-toolbox.formulabar.toolbox {
 	display: grid;
 	grid-auto-flow: column;
-	height: min-content;
+	align-items: center;
+	grid-column-gap: 4px;
+	grid-template-columns: repeat(3, 30px);
+}
+
+#formulabar-buttons-toolbox * {
+	align-items: center;
+	justify-content: center;
+}
+
+.unotoolbutton.formulabar {
+	display: flex;
 }
 
 .formulabar.unotoolbutton {
diff --git a/browser/css/spreadsheet.css b/browser/css/spreadsheet.css
index f0b49240e..0cfe17c1c 100644
--- a/browser/css/spreadsheet.css
+++ b/browser/css/spreadsheet.css
@@ -233,8 +233,6 @@
 #AutoSumMenu {
 	/* Make sure it matches buttons height*/
 	grid-template-rows: var(--btn-size);
-	/* Account for the arrow match width */
-	width: calc(var(--btn-size)*2);
 }
 
 #formulabar.expanded > .root-container > .formulabar {