fix(context-menu): fix context menu bugs

liujuping · JackLian · commit 8f0291fc3e58 · 2024-01-09T09:08:40.000+08:00
diff --git a/packages/designer/src/context-menu-actions.ts b/packages/designer/src/context-menu-actions.ts
@@ -17,6 +17,8 @@ export interface IContextMenuActions {
   adjustMenuLayout: IPublicApiMaterial['adjustContextMenuLayout'];
 }
 
+let destroyFn: Function | undefined;
+
 export class ContextMenuActions implements IContextMenuActions {
   actions: IPublicTypeContextMenuAction[] = [];
 
@@ -55,7 +57,7 @@ export class ContextMenuActions implements IContextMenuActions {
     const { bounds } = designer.project.simulator?.viewport || { bounds: { left: 0, top: 0 } };
     const { left: simulatorLeft, top: simulatorTop } = bounds;
 
-    let destroyFn: Function | undefined;
+    destroyFn?.();
 
     const destroy = () => {
       destroyFn?.();
diff --git a/packages/engine/src/inner-plugins/default-context-menu.ts b/packages/engine/src/inner-plugins/default-context-menu.ts
@@ -87,6 +87,9 @@ export const defaultContextMenu = (ctx: IPublicModelPluginContext) => {
       material.addContextMenuOption({
         name: 'copy',
         title: intl('Copy.1'),
+        condition(nodes) {
+          return nodes.length > 0;
+        },
         action(nodes) {
           if (!nodes || nodes.length < 1) {
             return;
@@ -159,6 +162,9 @@ export const defaultContextMenu = (ctx: IPublicModelPluginContext) => {
       material.addContextMenuOption({
         name: 'delete',
         title: intl('Delete'),
+        condition(nodes) {
+          return nodes.length > 0;
+        },
         action(nodes) {
           nodes.forEach((node) => {
             node.remove();
diff --git a/packages/utils/src/context-menu.scss b/packages/utils/src/context-menu.scss
@@ -1,6 +1,6 @@
 .context-menu-tree-wrap {
   position: relative;
-  padding: 4px 10px 4px 24px;
+  padding: 4px 10px 4px 32px;
 }
 
 .context-menu-tree-children {
diff --git a/packages/utils/src/context-menu.tsx b/packages/utils/src/context-menu.tsx
@@ -22,16 +22,8 @@ const Tree = (props: {
   const { node } = props;
 
   if (!node) {
-    return null;
-  }
-
-  if (!node.parent) {
     return (
-      <div className="context-menu-tree-wrap">
-        <div className="context-menu-tree-children">
-          {props.children}
-        </div>
-      </div>
+      <div className="context-menu-tree-wrap">{ props.children }</div>
     );
   }
 

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`.context-menu-tree-wrap {`
`2`	`2`	`position: relative;`
`3`		`- padding: 4px 10px 4px 24px;`
	`3`	`+ padding: 4px 10px 4px 32px;`
`4`	`4`	`}`
`5`	`5`
`6`	`6`	`.context-menu-tree-children {`