From b3b3b017556d61d26e0e868d87dc2d72a29a1f34 Mon Sep 17 00:00:00 2001
From: Micaela Goffe <micaela.goffe@neovici.se>
Date: Mon, 9 Dec 2024 11:37:40 +0100
Subject: [PATCH] fix(footer): set max-toolbar-items when mini table is used

---
 lib/render-footer.js   | 109 ++++++++++++++++++++---------------------
 lib/use-fast-layout.js |   8 ++-
 lib/use-footer.js      |  17 +++----
 lib/use-mini.js        |   1 +
 lib/use-omnitable.js   |   4 +-
 5 files changed, 70 insertions(+), 69 deletions(-)

diff --git a/lib/render-footer.js b/lib/render-footer.js
index 8720beee..d84ecbb8 100644
--- a/lib/render-footer.js
+++ b/lib/render-footer.js
@@ -11,61 +11,56 @@ export const renderFooter = ({
 	csvFilename,
 	xlsxFilename,
 	xlsxSheetname,
-	topPlacement
-}) => {
-	return html`
-		<cosmoz-bottom-bar
-			id="bottomBar"
-			?active=${!isEmpty(selectedItems.length)}
-		>
-			<slot name="info" slot="info">
-				${ngettext(
-					'{0} selected item',
-					'{0} selected items',
-					selectedItems.length
-				)}
-			</slot>
-			<slot name="actions" id="actions"></slot>
-			<!-- These slots are needed by cosmoz-bottom-bar
+	topPlacement,
+	maxToolbarItems,
+}) =>
+	html`<cosmoz-bottom-bar
+		id="bottomBar"
+		?active=${!isEmpty(selectedItems.length)}
+		?max-toolbar-items=${!isEmpty(maxToolbarItems)}
+	>
+		<slot name="info" slot="info">
+			${ngettext(
+				'{0} selected item',
+				'{0} selected items',
+				selectedItems.length,
+			)}
+		</slot>
+		<slot name="actions" id="actions"></slot>
+		<!-- These slots are needed by cosmoz-bottom-bar
         as it might change the slot of the actions to distribute them in the menu -->
-			<slot name="bottom-bar-toolbar" slot="bottom-bar-toolbar"></slot>
-			<slot name="bottom-bar-menu" slot="bottom-bar-menu"></slot>
-			<cosmoz-dropdown-menu slot="extra" .placement=${topPlacement}>
-				<svg
-					slot="button"
-					width="14"
-					height="18"
-					viewBox="0 0 14 18"
-					fill="none"
-					stroke="currentColor"
-					xmlns="http://www.w3.org/2000/svg"
-				>
-					<path
-						d="M1 8.5L7.00024 14.5L13 8.5"
-						stroke-width="2"
-						stroke-linecap="round"
-						stroke-linejoin="round"
-					/>
-					<path d="M13 17L1 17" stroke-width="2" stroke-linecap="round" />
-					<path d="M7 1V13" stroke-width="2" stroke-linecap="round" />
-				</svg>
-				<button
-					@click=${() => saveAsCsvAction(columns, selectedItems, csvFilename)}
-				>
-					${_('Save as CSV')}
-				</button>
-				<button
-					@click=${() =>
-						saveAsXlsxAction(
-							columns,
-							selectedItems,
-							xlsxFilename,
-							xlsxSheetname
-						)}
-				>
-					${_('Save as XLSX')}
-				</button>
-				<slot name="download-menu"></slot>
-			</cosmoz-dropdown-menu>
-		</cosmoz-bottom-bar>`;
-};
+		<slot name="bottom-bar-toolbar" slot="bottom-bar-toolbar"></slot>
+		<slot name="bottom-bar-menu" slot="bottom-bar-menu"></slot>
+		<cosmoz-dropdown-menu slot="extra" .placement=${topPlacement}>
+			<svg
+				slot="button"
+				width="14"
+				height="18"
+				viewBox="0 0 14 18"
+				fill="none"
+				stroke="currentColor"
+				xmlns="http://www.w3.org/2000/svg"
+			>
+				<path
+					d="M1 8.5L7.00024 14.5L13 8.5"
+					stroke-width="2"
+					stroke-linecap="round"
+					stroke-linejoin="round"
+				/>
+				<path d="M13 17L1 17" stroke-width="2" stroke-linecap="round" />
+				<path d="M7 1V13" stroke-width="2" stroke-linecap="round" />
+			</svg>
+			<button
+				@click=${() => saveAsCsvAction(columns, selectedItems, csvFilename)}
+			>
+				${_('Save as CSV')}
+			</button>
+			<button
+				@click=${() =>
+					saveAsXlsxAction(columns, selectedItems, xlsxFilename, xlsxSheetname)}
+			>
+				${_('Save as XLSX')}
+			</button>
+			<slot name="download-menu"></slot>
+		</cosmoz-dropdown-menu>
+	</cosmoz-bottom-bar>`;
diff --git a/lib/use-fast-layout.js b/lib/use-fast-layout.js
index 96e70b46..a316f496 100644
--- a/lib/use-fast-layout.js
+++ b/lib/use-fast-layout.js
@@ -16,7 +16,11 @@ export const useFastLayout = ({
 	sortAndGroupOptions,
 }) => {
 	const canvasWidth = useCanvasWidth(host),
-		{ miniColumn, miniColumns } = useMini({ host, canvasWidth, columns }),
+		{ isMiniSize, miniColumn, miniColumns } = useMini({
+			host,
+			canvasWidth,
+			columns,
+		}),
 		{ groupOnColumn } = sortAndGroupOptions,
 		layout = useLayout({
 			canvasWidth,
@@ -52,5 +56,5 @@ export const useFastLayout = ({
 
 	useStyleSheet(layoutCss);
 
-	return { collapsedColumns, miniColumns };
+	return { isMiniSize, collapsedColumns, miniColumns };
 };
diff --git a/lib/use-footer.js b/lib/use-footer.js
index a4d53571..56dc19f1 100644
--- a/lib/use-footer.js
+++ b/lib/use-footer.js
@@ -1,22 +1,21 @@
 import { defaultPlacement } from '@neovici/cosmoz-dropdown';
 const _defaultPlacement = ['top-right', ...defaultPlacement];
 
-export const useFooter = ({
-	host,
-	...rest
-}) => {
+export const useFooter = ({ host, ...rest }) => {
 	const {
-			csvFilename = 'omnitable.csv',
-			xlsxFilename = 'omnitable.xlsx',
-			xlsxSheetname = 'Omnitable',
-			topPlacement = _defaultPlacement,
-		} = host;
+		csvFilename = 'omnitable.csv',
+		xlsxFilename = 'omnitable.xlsx',
+		xlsxSheetname = 'Omnitable',
+		topPlacement = _defaultPlacement,
+		maxToolbarItems,
+	} = host;
 
 	return {
 		csvFilename,
 		xlsxFilename,
 		xlsxSheetname,
 		topPlacement,
+		maxToolbarItems,
 		...rest,
 	};
 };
diff --git a/lib/use-mini.js b/lib/use-mini.js
index cc2e64f4..9a41f216 100644
--- a/lib/use-mini.js
+++ b/lib/use-mini.js
@@ -23,6 +23,7 @@ export const useMini = ({ host, canvasWidth, columns: _columns }) => {
 	}, [miniColumn]);
 
 	return {
+		isMiniSize,
 		miniColumn,
 		miniColumns,
 	};
diff --git a/lib/use-omnitable.js b/lib/use-omnitable.js
index 64088bb9..5bf7c304 100644
--- a/lib/use-omnitable.js
+++ b/lib/use-omnitable.js
@@ -40,7 +40,7 @@ export const useOmnitable = (host) => {
 				noLocalSort,
 				noLocalFilter,
 			}),
-		{ collapsedColumns, miniColumns } = useFastLayout({
+		{ isMiniSize, collapsedColumns, miniColumns } = useFastLayout({
 			host,
 			columns,
 			settings,
@@ -48,6 +48,7 @@ export const useOmnitable = (host) => {
 			resizeSpeedFactor,
 			sortAndGroupOptions,
 		}),
+		maxToolbarItems = host.hasAttribute('mini') && isMiniSize ? '0' : '',
 		dataIsValid = data && Array.isArray(data) && data.length > 0,
 		[selectedItems, setSelectedItems] = useState([]);
 
@@ -94,6 +95,7 @@ export const useOmnitable = (host) => {
 			host,
 			selectedItems,
 			columns,
+			maxToolbarItems,
 		}),
 	};
 };