From 42eba03de72b5f22199737e459548db68b9ba809 Mon Sep 17 00:00:00 2001
From: Rupin Khandelwal <rupin@omnivore.app>
Date: Wed, 10 Aug 2022 18:59:01 +0200
Subject: [PATCH] adding text color variable so that it works for both themes
 for menu

---
 packages/web/components/templates/Menu.tsx |  9 ---------
 packages/web/styles/menu.css               | 22 ++++++++++++++--------
 2 files changed, 14 insertions(+), 17 deletions(-)

diff --git a/packages/web/components/templates/Menu.tsx b/packages/web/components/templates/Menu.tsx
index d586e6ce0..a21c707e4 100644
--- a/packages/web/components/templates/Menu.tsx
+++ b/packages/web/components/templates/Menu.tsx
@@ -27,9 +27,6 @@ const calculateTodayMenuItem = () => {
   const hrefStr = `?q=in%3Ainbox+saved%3A${
     new Date(new Date().getTime() - 24 * 3600000).toISOString().split('T')[0]
   }Z%${timeZoneHourDiff}B2..*`
-  //  console.log(timeZoneHourDiff.toLocaleString('en-US', {
-  //     signDisplay: 'always',
-  //   }))
   return hrefStr
 }
 
@@ -106,12 +103,6 @@ export const Menubar = () => {
       ...createDynamicMenuItems(labels, subscriptions),
     ])
   }, [labels, subscriptions])
-  //   const menuItems = useMemo(() => {
-  //     if (labels || subscriptions) {
-  //       setMenuList( [...menuList, ...createDynamicMenuItems(labels, subscriptions)])
-  //     }
-  //   }, [labels, subscriptions, menuList])
-
   return (
     <ProSidebar style={proSideBarStyles} breakPoint={'sm'}>
       <Menu>
diff --git a/packages/web/styles/menu.css b/packages/web/styles/menu.css
index 29c2af5c7..881117862 100644
--- a/packages/web/styles/menu.css
+++ b/packages/web/styles/menu.css
@@ -1,14 +1,20 @@
 /* Menu Override styles */
 
-.pro-sidebar > .pro-sidebar-inner {
+.pro-sidebar>.pro-sidebar-inner {
     background-color: var(--colors-grayBase);
-    color: #000;
-  }
-
-  .pro-sidebar .pro-menu a {
-    color: #000;
+    color: var(--colors-utilityTextDefault);
 }
 
-.pro-sidebar .pro-menu .pro-menu-item > .pro-inner-item {
-    padding: 5px 30px 5px 20px;
+.pro-sidebar .pro-menu a {
+    color: var(--colors-utilityTextDefault);
+}
+
+.pro-sidebar .pro-menu .pro-menu-item>.pro-inner-item {
+    padding: 3px 20px 3px 20px;
+}
+
+.pro-sidebar .pro-menu .pro-menu-item>.pro-inner-item>.pro-icon-wrapper {
+    width: 20px;
+    min-width: 0;
+    height: 25px;
 }
\ No newline at end of file