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

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


The following commit(s) were added to refs/heads/master by this push:
     new 11b49a6ceb fix(SqlLab): make icon placement even (#25372)
11b49a6ceb is described below

commit 11b49a6ceb7b258766d7fe4642808509ccf83317
Author: Corbin Bullard <[email protected]>
AuthorDate: Thu Sep 28 09:36:24 2023 -0400

    fix(SqlLab): make icon placement even (#25372)
---
 superset-frontend/src/SqlLab/components/SqlEditorLeftBar/index.tsx | 4 +++-
 superset-frontend/src/SqlLab/components/TableElement/index.tsx     | 5 +++++
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/superset-frontend/src/SqlLab/components/SqlEditorLeftBar/index.tsx 
b/superset-frontend/src/SqlLab/components/SqlEditorLeftBar/index.tsx
index 169acd82ce..ef55e30ec6 100644
--- a/superset-frontend/src/SqlLab/components/SqlEditorLeftBar/index.tsx
+++ b/superset-frontend/src/SqlLab/components/SqlEditorLeftBar/index.tsx
@@ -85,7 +85,9 @@ const collapseStyles = (theme: SupersetTheme) => css`
     padding: 0px ${theme.gridUnit * 4}px 0px 0px !important;
   }
   .ant-collapse-arrow {
-    top: ${theme.gridUnit * 2}px !important;
+    padding: 0 !important;
+    bottom: ${theme.gridUnit}px !important;
+    right: ${theme.gridUnit * 4}px !important;
     color: ${theme.colors.primary.dark1} !important;
     &:hover {
       color: ${theme.colors.primary.dark2} !important;
diff --git a/superset-frontend/src/SqlLab/components/TableElement/index.tsx 
b/superset-frontend/src/SqlLab/components/TableElement/index.tsx
index 979b9d85e9..7a21331d99 100644
--- a/superset-frontend/src/SqlLab/components/TableElement/index.tsx
+++ b/superset-frontend/src/SqlLab/components/TableElement/index.tsx
@@ -259,6 +259,11 @@ const TableElement = ({ table, ...props }: 
TableElementProps) => {
           display: flex;
           column-gap: ${theme.gridUnit * 1.5}px;
           margin-right: ${theme.gridUnit}px;
+          & span {
+            display: flex;
+            justify-content: center;
+            width: ${theme.gridUnit * 4}px;
+          }
         `}
       >
         {keyLink}

Reply via email to