From 19ef69ef6e5308e39583565aa0a1e59400934b17 Mon Sep 17 00:00:00 2001 From: Diego Medina Date: Mon, 25 Apr 2022 16:25:49 -0300 Subject: [PATCH] fix: Regression on Data and Alerts & Reports Headers --- .../src/views/components/SubMenu.tsx | 15 ++++++++++++--- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/superset-frontend/src/views/components/SubMenu.tsx b/superset-frontend/src/views/components/SubMenu.tsx index 2ae897196b78b..3ac1253709379 100644 --- 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;