fix: Regression on Data and Alerts & Reports Headers (apache#19850)

ValtechMobility · Jun 9, 2022 · bee364b · bee364b
1 parent ef5595b
commit bee364b
Showing 1 changed file with 12 additions and 3 deletions.
diff --git a/superset-frontend/src/views/components/SubMenu.tsx b/superset-frontend/src/views/components/SubMenu.tsx
@@ -67,7 +67,7 @@ const StyledHeader = styled.div`
     padding-left: 10px;
   }
   .menu {
-    background-color: white;
+    background-color: ${({ theme }) => theme.colors.grayscale.light5};
     .ant-menu-horizontal {
       line-height: inherit;
       .ant-menu-item {
@@ -88,7 +88,8 @@ const StyledHeader = styled.div`
   }
 
   .menu .ant-menu-item {
-    li {
+    li,
+    div {
       a,
       div {
         font-size: ${({ theme }) => theme.typography.sizes.s}px;
@@ -98,6 +99,10 @@ const StyledHeader = styled.div`
           margin: 0;
           padding: ${({ theme }) => theme.gridUnit * 4}px;
           line-height: ${({ theme }) => theme.gridUnit * 5}px;
+
+          &:hover {
+            text-decoration: none;
+          }
         }
       }
 
@@ -106,11 +111,14 @@ const StyledHeader = styled.div`
           ${({ theme }) => theme.gridUnit * 4}px;
       }
     }
+
     li.active > a,
     li.active > div,
+    div.active > div,
     li > a:hover,
     li > a:focus,
-    li > div:hover {
+    li > div:hover,
+    div > div:hover {
       background: ${({ theme }) => theme.colors.secondary.light4};
       border-bottom: none;
       border-radius: ${({ theme }) => theme.borderRadius}px;
@@ -148,6 +156,7 @@ const StyledHeader = styled.div`
 const styledDisabled = (theme: SupersetTheme) => css`
   color: ${theme.colors.grayscale.base};
   backgroundColor: ${theme.colors.grayscale.light2}};
+
   .ant-menu-item:hover {
     color: ${theme.colors.grayscale.base};
     cursor: default;