Don’t expand the individual node if you’re holding alt

matt-aitken · matt-aitken · commit 3a550b5e884c · 2024-04-22T13:32:15.000+01:00
diff --git a/apps/webapp/app/components/primitives/TreeView/TreeView.tsx b/apps/webapp/app/components/primitives/TreeView/TreeView.tsx
@@ -434,6 +434,8 @@ export function useTree<TData>({
           }
           case "Right":
           case "ArrowRight": {
+            e.preventDefault();
+
             const selected = selectedIdFromState(state.nodes);
 
             if (selected) {
@@ -443,11 +445,11 @@ export function useTree<TData>({
                 if (treeNode && treeNode.hasChildren) {
                   expandLevel(treeNode.level);
                 }
+                break;
               }
 
               expandNode(selected, true);
             }
-            e.preventDefault();
             break;
           }
           case "Escape": {

Original file line number	Diff line number	Diff line change
`@@ -434,6 +434,8 @@ export function useTree<TData>({`
`434`	`434`	`}`
`435`	`435`	`case "Right":`
`436`	`436`	`case "ArrowRight": {`
	`437`	`+ e.preventDefault();`
	`438`	`+`
`437`	`439`	`const selected = selectedIdFromState(state.nodes);`
`438`	`440`
`439`	`441`	`if (selected) {`
`@@ -443,11 +445,11 @@ export function useTree<TData>({`
`443`	`445`	`if (treeNode && treeNode.hasChildren) {`
`444`	`446`	`expandLevel(treeNode.level);`
`445`	`447`	`}`
	`448`	`+ break;`
`446`	`449`	`}`
`447`	`450`
`448`	`451`	`expandNode(selected, true);`
`449`	`452`	`}`
`450`		`- e.preventDefault();`
`451`	`453`	`break;`
`452`	`454`	`}`
`453`	`455`	`case "Escape": {`