From cd1925595a11d21232c35da447dc2b2c74b8a77c Mon Sep 17 00:00:00 2001
From: Ruben Fiszel <ruben@rubenfiszel.com>
Date: Sat, 13 Jan 2024 15:38:59 +0100
Subject: [PATCH] fix: menu colors

---
 .../lib/components/sidebar/MenuButton.svelte  | 21 +++++++++++++++----
 1 file changed, 17 insertions(+), 4 deletions(-)

diff --git a/frontend/src/lib/components/sidebar/MenuButton.svelte b/frontend/src/lib/components/sidebar/MenuButton.svelte
index bdbaec2bf1..2708a70860 100644
--- a/frontend/src/lib/components/sidebar/MenuButton.svelte
+++ b/frontend/src/lib/components/sidebar/MenuButton.svelte
@@ -13,8 +13,10 @@
 	<Popover appearTimeout={0} disappearTimeout={0} class="w-full" disablePopup={!isCollapsed}>
 		<button
 			class={twMerge(
-				'group flex items-center px-2 py-2 font-light rounded-md h-8 gap-3 w-full text-primary ',
-				lightMode ? 'hover:bg-surface-hover ' : '  hover:bg-[#34363c]',
+				'group flex items-center px-2 py-2 font-light rounded-md h-8 gap-3 w-full',
+				lightMode
+					? 'text-primary hover:bg-surface-hover '
+					: '  hover:bg-[#34363c] text-primary-inverse dark:text-primary',
 				'transition-all',
 				$$props.class
 			)}
@@ -24,13 +26,24 @@
 				<svelte:component
 					this={icon}
 					size={16}
-					class={twMerge('flex-shrink-0 text-primary group-hover:text-secondary', 'transition-all')}
+					class={twMerge(
+						'flex-shrink-0',
+						lightMode
+							? 'text-primary group-hover:text-secondary'
+							: 'text-primary-inverse group-hover:text-secondary-inverse dark:group-hover:text-secondary dark:text-primary',
+						'transition-all'
+					)}
 				/>
 			{/if}
 
 			{#if !isCollapsed && label}
 				<span
-					class={twMerge('whitespace-pre truncate text-primary', 'transition-all', $$props.class)}
+					class={twMerge(
+						'whitespace-pre truncate',
+						lightMode ? 'text-primary' : 'text-primary-inverse  dark:text-primary',
+						'transition-all',
+						$$props.class
+					)}
 				>
 					{label}
 				</span>