MINOR: sidebar ui fix (open-metadata#14761)

* MINOR: sidebar ui fix * fix app icon movement
gorgias · Jan 17, 2024 · 0d7833b · 0d7833b
1 parent 0d0d820
commit 0d7833b
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 17 deletions.
diff --git a/...data-ui/src/main/resources/ui/src/components/MyData/LeftSidebar/LeftSidebar.component.tsx b/...data-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
@@ -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;