fix: fix the problem of using canvas.dragon api in workspace mode

liujuping · JackLian · commit b07d33e99a14 · 2023-01-05T11:28:03.000+08:00
diff --git a/packages/designer/src/designer/designer.ts b/packages/designer/src/designer/designer.ts
@@ -59,7 +59,7 @@ export interface DesignerProps {
 
 
 export class Designer implements IDesigner {
-  readonly dragon = new Dragon(this);
+  dragon: Dragon;
 
   readonly componentActions = new ComponentActions();
 
@@ -99,6 +99,7 @@ export class Designer implements IDesigner {
 
     this.project = new Project(this, props.defaultSchema, viewName);
 
+    this.dragon = new Dragon(this);
     this.dragon.onDragstart((e) => {
       this.detecting.enable = false;
       const { dragObject } = e;
@@ -564,6 +565,10 @@ export class Designer implements IDesigner {
   }
 
   addPropsReducer(reducer: IPublicTypePropsTransducer, stage: IPublicEnumTransformStage) {
+    if (!reducer) {
+      logger.error('reducer is not available');
+      return;
+    }
     const reducers = this.propsReducers.get(stage);
     if (reducers) {
       reducers.push(reducer);
diff --git a/packages/shell/src/api/canvas.ts b/packages/shell/src/api/canvas.ts
@@ -26,7 +26,7 @@ export class Canvas implements IPublicApiCanvas {
     return this[editorSymbol].get('designer') as IDesigner;
   }
 
-  constructor(editor: IPublicModelEditor) {
+  constructor(editor: IPublicModelEditor, readonly workspaceMode: boolean = false) {
     this[editorSymbol] = editor;
   }
 
@@ -49,7 +49,7 @@ export class Canvas implements IPublicApiCanvas {
   }
 
   get dragon(): IPublicModelDragon | null {
-    return Dragon.create(this[designerSymbol].dragon);
+    return Dragon.create(this[designerSymbol].dragon, this.workspaceMode);
   }
 
   get activeTracker(): IPublicModelActiveTracker | null {
diff --git a/packages/shell/src/model/dragon.ts b/packages/shell/src/model/dragon.ts
@@ -19,11 +19,14 @@ export const innerDragonSymbol = Symbol('innerDragonSymbol');
 export class Dragon implements IPublicModelDragon {
   private readonly [innerDragonSymbol]: IPublicModelDragon;
 
-  constructor(innerDragon: IPublicModelDragon) {
+  constructor(innerDragon: IPublicModelDragon, readonly workspaceMode: boolean) {
     this[innerDragonSymbol] = innerDragon;
   }
 
   get [dragonSymbol](): any {
+    if (this.workspaceMode) {
+      return this[innerDragonSymbol];
+    }
     const workspace = globalContext.get('workspace');
     let editor = globalContext.get('editor');
 
@@ -35,11 +38,11 @@ export class Dragon implements IPublicModelDragon {
     return designer.dragon;
   }
 
-  static create(dragon: IPublicModelDragon | null): IPublicModelDragon | null {
+  static create(dragon: IPublicModelDragon | null, workspaceMode: boolean): IPublicModelDragon | null {
     if (!dragon) {
       return null;
     }
-    return new Dragon(dragon);
+    return new Dragon(dragon, workspaceMode);
   }
 
   /**
diff --git a/packages/workspace/src/base-context.ts b/packages/workspace/src/base-context.ts
@@ -81,7 +81,7 @@ export class BasicContext {
     const event = new Event(commonEvent, { prefix: 'common' });
     const logger = getLogger({ level: 'warn', bizName: 'common' });
     const skeleton = new Skeleton(innerSkeleton, 'any', true);
-    const canvas = new Canvas(editor);
+    const canvas = new Canvas(editor, true);
     editor.set('setters', setters);
     editor.set('project', project);
     editor.set('material', material);
diff --git a/packages/workspace/src/layouts/workbench.less b/packages/workspace/src/layouts/workbench.less
@@ -153,6 +153,7 @@ body {
     .lc-top-area-left, .lc-sub-top-area-left {
       display: flex;
       align-items: center;
+      max-width: 100%;
     }
 
     .lc-top-area-center, .lc-sub-top-area-center {
@@ -361,6 +362,18 @@ body {
       }
     }
 
+    .lc-workspace-workbench-center-content {
+      position: absolute;
+      top: 0;
+      right: 0;
+      bottom: 0;
+      left: 0;
+    }
+
+    .engine-actionitem {
+      max-width: 100%;
+    }
+
     .lc-workspace-workbench-window {
       position: relative;
       height: 100%;
diff --git a/packages/workspace/src/layouts/workbench.tsx b/packages/workspace/src/layouts/workbench.tsx
@@ -41,7 +41,7 @@ export class Workbench extends Component<{
             <LeftFloatPane area={skeleton.leftFloatArea} />
             <LeftFixedPane area={skeleton.leftFixedArea} />
             <div className="lc-workspace-workbench-center">
-              <>
+              <div className="lc-workspace-workbench-center-content">
                 <SubTopArea area={skeleton.subTopArea} itemClassName={topAreaItemClassName} />
                 <div className="lc-workspace-workbench-window">
                   {
@@ -54,7 +54,7 @@ export class Workbench extends Component<{
                     ))
                   }
                 </div>
-              </>
+              </div>
               <MainArea area={skeleton.mainArea} />
               <BottomArea area={skeleton.bottomArea} />
             </div>
diff --git a/packages/workspace/src/workspace.ts b/packages/workspace/src/workspace.ts
@@ -101,7 +101,7 @@ export class Workspace implements IPublicApiWorkspace {
 
   private remove(index: number) {
     const window = this.windows[index];
-    this.windows = this.windows.splice(index - 1, 1);
+    this.windows.splice(index, 1);
     if (this.window === window) {
       this.window = this.windows[index] || this.windows[index + 1] || this.windows[index - 1];
       this.emitChangeActiveWindow();

Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ export class Canvas implements IPublicApiCanvas {`
`26`	`26`	`return this[editorSymbol].get('designer') as IDesigner;`
`27`	`27`	`}`
`28`	`28`
`29`		`- constructor(editor: IPublicModelEditor) {`
	`29`	`+ constructor(editor: IPublicModelEditor, readonly workspaceMode: boolean = false) {`
`30`	`30`	`this[editorSymbol] = editor;`
`31`	`31`	`}`
`32`	`32`
`@@ -49,7 +49,7 @@ export class Canvas implements IPublicApiCanvas {`
`49`	`49`	`}`
`50`	`50`
`51`	`51`	`get dragon(): IPublicModelDragon \| null {`
`52`		`- return Dragon.create(this[designerSymbol].dragon);`
	`52`	`+ return Dragon.create(this[designerSymbol].dragon, this.workspaceMode);`
`53`	`53`	`}`
`54`	`54`
`55`	`55`	`get activeTracker(): IPublicModelActiveTracker \| null {`
Original file line number	Diff line number	Diff line change
`@@ -19,11 +19,14 @@ export const innerDragonSymbol = Symbol('innerDragonSymbol');`
`19`	`19`	`export class Dragon implements IPublicModelDragon {`
`20`	`20`	`private readonly [innerDragonSymbol]: IPublicModelDragon;`
`21`	`21`
`22`		`- constructor(innerDragon: IPublicModelDragon) {`
	`22`	`+ constructor(innerDragon: IPublicModelDragon, readonly workspaceMode: boolean) {`
`23`	`23`	`this[innerDragonSymbol] = innerDragon;`
`24`	`24`	`}`
`25`	`25`
`26`	`26`	`get [dragonSymbol](): any {`
	`27`	`+ if (this.workspaceMode) {`
	`28`	`+ return this[innerDragonSymbol];`
	`29`	`+ }`
`27`	`30`	`const workspace = globalContext.get('workspace');`
`28`	`31`	`let editor = globalContext.get('editor');`
`29`	`32`
`@@ -35,11 +38,11 @@ export class Dragon implements IPublicModelDragon {`
`35`	`38`	`return designer.dragon;`
`36`	`39`	`}`
`37`	`40`
`38`		`- static create(dragon: IPublicModelDragon \| null): IPublicModelDragon \| null {`
	`41`	`+ static create(dragon: IPublicModelDragon \| null, workspaceMode: boolean): IPublicModelDragon \| null {`
`39`	`42`	`if (!dragon) {`
`40`	`43`	`return null;`
`41`	`44`	`}`
`42`		`- return new Dragon(dragon);`
	`45`	`+ return new Dragon(dragon, workspaceMode);`
`43`	`46`	`}`
`44`	`47`
`45`	`48`	`/**`