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 b7a3224f040 feat: Larger folder drag area in folders editor (#38102)
b7a3224f040 is described below

commit b7a3224f0405da18c2c29b4f0f09d284630112d1
Author: Kamil Gabryjelski <[email protected]>
AuthorDate: Thu Feb 19 18:22:04 2026 +0100

    feat: Larger folder drag area in folders editor (#38102)
---
 .../Datasource/FoldersEditor/TreeItem.styles.ts           | 15 ++++++---------
 .../src/components/Datasource/FoldersEditor/TreeItem.tsx  |  5 +++--
 2 files changed, 9 insertions(+), 11 deletions(-)

diff --git 
a/superset-frontend/src/components/Datasource/FoldersEditor/TreeItem.styles.ts 
b/superset-frontend/src/components/Datasource/FoldersEditor/TreeItem.styles.ts
index b28fa0fe985..d7d678a999a 100644
--- 
a/superset-frontend/src/components/Datasource/FoldersEditor/TreeItem.styles.ts
+++ 
b/superset-frontend/src/components/Datasource/FoldersEditor/TreeItem.styles.ts
@@ -83,6 +83,10 @@ export const TreeFolderContainer = 
styled(TreeItemContainer)<{
     margin-right: ${theme.marginMD}px;
     transition: background-color 0.15s ease-in-out, box-shadow 0.15s 
ease-in-out;
 
+    &:hover:not(:has(input)) [aria-label="move"] {
+      color: ${theme.colorText};
+    }
+
     /* Drop target styles - controlled via data attributes for performance */
     &[data-drop-target="true"] {
       background-color: ${theme.colorPrimaryBg};
@@ -116,15 +120,7 @@ export const DragHandle = styled.span`
     color: ${theme.colorTextTertiary};
     display: inline-flex;
     align-items: center;
-    cursor: grab;
-
-    &:hover {
-      color: ${theme.colorText};
-    }
-
-    &:active {
-      cursor: grabbing;
-    }
+    transition: color 0.15s ease-in-out;
   `}
 `;
 
@@ -158,6 +154,7 @@ export const FolderName = styled.span`
   ${({ theme }) => `
     margin-right: ${theme.marginMD}px;
     font-weight: ${theme.fontWeightStrong};
+    cursor: pointer;
   `}
 `;
 
diff --git 
a/superset-frontend/src/components/Datasource/FoldersEditor/TreeItem.tsx 
b/superset-frontend/src/components/Datasource/FoldersEditor/TreeItem.tsx
index a941fd19474..11e4c31c59e 100644
--- a/superset-frontend/src/components/Datasource/FoldersEditor/TreeItem.tsx
+++ b/superset-frontend/src/components/Datasource/FoldersEditor/TreeItem.tsx
@@ -262,8 +262,6 @@ function TreeItemComponent({
     <>
       {isFolder && (
         <DragHandle
-          {...attributes}
-          {...listeners}
           css={theme => css`
             margin-right: ${theme.marginSM}px;
           `}
@@ -348,9 +346,12 @@ function TreeItemComponent({
         {isFolder ? (
           <TreeFolderContainer
             {...restContainerProps}
+            {...attributes}
+            {...listeners}
             data-folder-id={id}
             data-drop-target={isDropTarget ? 'true' : undefined}
             isForbiddenDropTarget={isForbiddenDrop}
+            css={{ cursor: 'grab', '&:active': { cursor: 'grabbing' } }}
           >
             {containerContent}
           </TreeFolderContainer>

Reply via email to