From 0d7833bf234539cf68a1b8affe1bf1d425dd616e Mon Sep 17 00:00:00 2001
From: Ashish Gupta <ashish@getcollate.io>
Date: Wed, 17 Jan 2024 21:49:21 +0530
Subject: [PATCH] MINOR: sidebar ui fix (#14761)

* MINOR: sidebar ui fix

* fix app icon movement
---
 .../LeftSidebar/LeftSidebar.component.tsx     |  4 +--
 .../MyData/LeftSidebar/left-sidebar.less      | 25 ++++++++-----------
 2 files changed, 12 insertions(+), 17 deletions(-)

diff --git a/openmetadata-ui/src/main/resources/ui/src/components/MyData/LeftSidebar/LeftSidebar.component.tsx b/openmetadata-ui/src/main/resources/ui/src/components/MyData/LeftSidebar/LeftSidebar.component.tsx
index 6a1833009e09..2c07709578a3 100644
--- a/openmetadata-ui/src/main/resources/ui/src/components/MyData/LeftSidebar/LeftSidebar.component.tsx
+++ b/openmetadata-ui/src/main/resources/ui/src/components/MyData/LeftSidebar/LeftSidebar.component.tsx
@@ -107,9 +107,7 @@ const LeftSidebar = () => {
           <Link className="flex-shrink-0" id="openmetadata_logo" to="/">
             <BrandImage
               alt="OpenMetadata Logo"
-              className={classNames('vertical-middle ', {
-                'm-l-sm': !isSidebarCollapsed,
-              })}
+              className="vertical-middle"
               dataTestId="image"
               height={isSidebarCollapsed ? 30 : 34}
               isMonoGram={isSidebarCollapsed}
diff --git a/openmetadata-ui/src/main/resources/ui/src/components/MyData/LeftSidebar/left-sidebar.less b/openmetadata-ui/src/main/resources/ui/src/components/MyData/LeftSidebar/left-sidebar.less
index 1d7bc1e5a1cb..02fdebddb9c2 100644
--- a/openmetadata-ui/src/main/resources/ui/src/components/MyData/LeftSidebar/left-sidebar.less
+++ b/openmetadata-ui/src/main/resources/ui/src/components/MyData/LeftSidebar/left-sidebar.less
@@ -42,9 +42,8 @@
 
   .brand-logo-container {
     height: 64px;
-    display: flex;
-    align-items: center;
-    justify-content: center;
+    padding-top: 17px;
+    padding-left: 14px;
     background: rgba(248, 248, 248, 1);
   }
 
@@ -53,10 +52,6 @@
   .sidebar-open {
     width: 230px;
 
-    .brand-logo-container {
-      justify-content: flex-start;
-    }
-
     .ant-menu-submenu-arrow {
       opacity: 1;
     }
@@ -69,12 +64,6 @@
       display: none;
     }
 
-    .ant-menu-submenu-open {
-      .ant-menu-submenu-title {
-        background: rgba(9, 80, 197, 0.04);
-      }
-    }
-
     .left-sidebar-menu.ant-menu {
       .ant-menu-item {
         .ant-menu-item-icon {
@@ -233,7 +222,8 @@
     &:not(.ant-menu-horizontal) .ant-menu-item-selected,
     .ant-menu-item:active,
     .ant-menu-submenu-title:active,
-    .ant-menu-item:hover {
+    .ant-menu-item:hover,
+    .ant-menu-submenu-title:hover {
       background-color: @left-nav-item-background;
       color: @text-color;
 
@@ -266,6 +256,13 @@
   text-decoration: none;
 }
 
+.left-panel-item.ant-btn-text {
+  &:hover,
+  &:focus {
+    background: initial;
+  }
+}
+
 .page-layout-leftpanel {
   padding: 0 0 24px 0 !important;
   border-right: 1px solid @border-color;