You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by vi...@apache.org on 2021/01/25 13:12:04 UTC

[superset] 28/38: [12601] Hovered menu items on dashboard - brought back padding and added margin on top of chart (#12603)

This is an automated email from the ASF dual-hosted git repository.

villebro pushed a commit to branch 1.0
in repository https://gitbox.apache.org/repos/asf/superset.git

commit 3d770aeeb945d1ce1aeed3e0109ff106c80d02f8
Author: Kasia Kucharczyk <25...@users.noreply.github.com>
AuthorDate: Thu Jan 21 10:32:34 2021 +0100

    [12601] Hovered menu items on dashboard - brought back padding and added margin on top of chart (#12603)
---
 superset-frontend/src/dashboard/components/DashboardBuilder.jsx | 2 +-
 superset-frontend/src/dashboard/stylesheets/hover-menu.less     | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/superset-frontend/src/dashboard/components/DashboardBuilder.jsx b/superset-frontend/src/dashboard/components/DashboardBuilder.jsx
index aad5c41..2c9ba0c 100644
--- a/superset-frontend/src/dashboard/components/DashboardBuilder.jsx
+++ b/superset-frontend/src/dashboard/components/DashboardBuilder.jsx
@@ -93,7 +93,7 @@ const StyledDashboardContent = styled.div`
     width: 100%;
     flex-grow: 1;
     position: relative;
-    margin: ${({ theme }) => theme.gridUnit * 2}px
+    margin: ${({ theme }) => theme.gridUnit * 6}px
       ${({ theme }) => theme.gridUnit * 8}px
       ${({ theme }) => theme.gridUnit * 6}px
       ${({ theme, dashboardFiltersOpen }) => {
diff --git a/superset-frontend/src/dashboard/stylesheets/hover-menu.less b/superset-frontend/src/dashboard/stylesheets/hover-menu.less
index 8953ce0..41ce575 100644
--- a/superset-frontend/src/dashboard/stylesheets/hover-menu.less
+++ b/superset-frontend/src/dashboard/stylesheets/hover-menu.less
@@ -28,6 +28,7 @@
   top: 50%;
   transform: translate(0, -50%);
   left: -28px;
+  padding: 8px 0;
   display: flex;
   flex-direction: column;
   justify-content: center;