@univerjs/sheets-drawing-ui 0.2.9 → 0.2.10

package/lib/es/index.js

@@ -121,7 +121,7 @@ let DrawingPopupMenuController = (_a = class extends RxDisposable {
     this._renderManagerService.has(unitId) && !this._initImagePopupMenu.has(unitId) && (this._popupMenuListener(unitId), this._initImagePopupMenu.add(unitId));
   }
   _hasCropObject(scene) {
-    const objects = scene.getAllObjects();
+    const objects = scene.getAllObjectsByOrder();
     for (const object of objects)
       if (object instanceof ImageCropperObject)
         return !0;
@@ -473,7 +473,7 @@ let SheetDrawingUpdateController = (_a2 = class extends Disposable {
   constructor(_context, _skeletonManagerService, _commandService, _selectionRenderService, _imageIoService, _sheetDrawingService, _drawingManagerService, _contextService, _messageService, _localeService, selectionManagerService) {
     super();
     __publicField(this, "_workbookSelections");
-    this._context = _context, this._skeletonManagerService = _skeletonManagerService, this._commandService = _commandService, this._selectionRenderService = _selectionRenderService, this._imageIoService = _imageIoService, this._sheetDrawingService = _sheetDrawingService, this._drawingManagerService = _drawingManagerService, this._contextService = _contextService, this._messageService = _messageService, this._localeService = _localeService, this._workbookSelections = selectionManagerService.getWorkbookSelections(this._context.unitId), this._initCommandListeners(), this._updateImageListener(), this._updateOrderListener(), this._groupDrawingListener(), this._focusDrawingListener(), this._drawingAddListener();
+    this._context = _context, this._skeletonManagerService = _skeletonManagerService, this._commandService = _commandService, this._selectionRenderService = _selectionRenderService, this._imageIoService = _imageIoService, this._sheetDrawingService = _sheetDrawingService, this._drawingManagerService = _drawingManagerService, this._contextService = _contextService, this._messageService = _messageService, this._localeService = _localeService, this._workbookSelections = selectionManagerService.getWorkbookSelections(this._context.unitId), this._initCommandListeners(), this._updateImageListener(), this._updateOrderListener(), this._groupDrawingListener(), this._focusDrawingListener();
   }
   /**
    * Upload image to cell or float image
@@ -648,24 +648,6 @@ let SheetDrawingUpdateController = (_a2 = class extends Disposable {
       })
     );
   }
-  _drawingAddListener() {
-    this.disposeWithMe(
-      this._sheetDrawingService.add$.subscribe((params) => {
-        this._registerDrawing(params);
-      })
-    );
-  }
-  _registerDrawing(params) {
-    params.forEach((param) => {
-      const drawingParam = this._sheetDrawingService.getDrawingByParam(param);
-      if (drawingParam == null || drawingParam.unitId !== this._context.unitId)
-        return;
-      const { sheetTransform } = drawingParam;
-      drawingParam.transform = drawingPositionToTransform(sheetTransform, this._selectionRenderService, this._skeletonManagerService);
-    });
-    const unitId = params[0].unitId;
-    this._drawingManagerService.registerDrawingData(unitId, this._sheetDrawingService.getDrawingDataForUnit(unitId)), this._drawingManagerService.initializeNotification(unitId);
-  }
 }, __name(_a2, "SheetDrawingUpdateController"), _a2);
 SheetDrawingUpdateController = __decorateClass$8([
   __decorateParam$8(1, Inject(SheetSkeletonManagerService)),
@@ -2148,7 +2130,7 @@ let SheetDrawingPermissionController = (_a7 = class extends Disposable {
             const unitId2 = workbook.getUnitId(), subUnitId2 = sheet.getSheetId(), drawingData = this._drawingManagerService.getDrawingData(unitId2, subUnitId2), drawingDataValues = Object.values(drawingData), renderObject = this._renderManagerService.getRenderById(unitId2), scene = renderObject == null ? void 0 : renderObject.scene;
             if (scene == null)
               return;
-            scene.getAllObjects().forEach((object) => {
+            scene.getAllObjectsByOrder().forEach((object) => {
               object.classType === RENDER_CLASS_TYPE.IMAGE && drawingDataValues.some((item) => object.oKey.includes(item.drawingId)) && scene.removeObject(object);
             });
           }
@@ -2177,7 +2159,7 @@ let SheetDrawingPermissionController = (_a7 = class extends Disposable {
             const unitId2 = workbook.getUnitId(), subUnitId2 = sheet.getSheetId(), drawingData = this._drawingManagerService.getDrawingData(unitId2, subUnitId2), drawingDataValues = Object.values(drawingData), renderObject = this._renderManagerService.getRenderById(unitId2), scene = renderObject == null ? void 0 : renderObject.scene;
             if (scene == null)
               return;
-            scene.getAllObjects().forEach((object) => {
+            scene.getAllObjectsByOrder().forEach((object) => {
               object.classType === RENDER_CLASS_TYPE.IMAGE && drawingDataValues.some((item) => object.oKey.includes(item.drawingId)) && scene.detachTransformerFrom(object);
             });
           }
@@ -2205,7 +2187,7 @@ let SheetDrawingPermissionController = (_a7 = class extends Disposable {
           ).subscribe({
             next: /* @__PURE__ */ __name((permission) => {
               initialViewPermission = permission, this._drawingManagerService.setDrawingVisible(permission);
-              const objects = scene.getAllObjects(), drawingData = this._drawingManagerService.getDrawingData(unitId, subUnitId), drawingDataValues = Object.values(drawingData);
+              const objects = scene.getAllObjectsByOrder(), drawingData = this._drawingManagerService.getDrawingData(unitId, subUnitId), drawingDataValues = Object.values(drawingData);
               permission ? this._drawingManagerService.addNotification(drawingDataValues) : (objects.forEach((object) => {
                 object.classType === RENDER_CLASS_TYPE.IMAGE && drawingDataValues.some((item) => object.oKey.includes(item.drawingId)) && scene.removeObject(object);
               }), transformer.clearSelectedObjects());
@@ -2244,7 +2226,7 @@ let SheetDrawingPermissionController = (_a7 = class extends Disposable {
           ).subscribe({
             next: /* @__PURE__ */ __name((permission) => {
               initialEditPermission = permission, this._drawingManagerService.setDrawingEditable(permission);
-              const objects = scene.getAllObjects(), drawingData = this._drawingManagerService.getDrawingData(unitId, subUnitId), drawingDataValues = Object.values(drawingData);
+              const objects = scene.getAllObjectsByOrder(), drawingData = this._drawingManagerService.getDrawingData(unitId, subUnitId), drawingDataValues = Object.values(drawingData);
               permission ? (objects.forEach((object) => {
                 object.classType === RENDER_CLASS_TYPE.IMAGE && drawingDataValues.some((item) => object.oKey.includes(item.drawingId)) && scene.attachTransformerTo(object);
               }), this._drawingManagerService.addNotification(drawingDataValues)) : (objects.forEach((object) => {
@@ -2260,7 +2242,7 @@ let SheetDrawingPermissionController = (_a7 = class extends Disposable {
               const unitId2 = workbook.getUnitId(), subUnitId2 = sheet.getSheetId(), drawingData = this._drawingManagerService.getDrawingData(unitId2, subUnitId2), drawingDataValues = Object.values(drawingData), renderObject2 = this._renderManagerService.getRenderById(unitId2), scene2 = renderObject2 == null ? void 0 : renderObject2.scene;
               if (scene2 == null)
                 return;
-              this._drawingManagerService.setDrawingEditable(!0), scene2.getAllObjects().forEach((object) => {
+              this._drawingManagerService.setDrawingEditable(!0), scene2.getAllObjectsByOrder().forEach((object) => {
                 object.classType === RENDER_CLASS_TYPE.IMAGE && drawingDataValues.some((item) => object.oKey.includes(item.drawingId)) && scene2.detachTransformerFrom(object);
               });
             }, "complete")
@@ -2421,21 +2403,22 @@ var __defProp$1 = Object.defineProperty, __getOwnPropDesc$1 = Object.getOwnPrope
   return kind && result && __defProp$1(target, key, result), result;
 }, "__decorateClass$1"), __decorateParam$1 = /* @__PURE__ */ __name((index, decorator) => (target, key) => decorator(target, key, index), "__decorateParam$1"), _a9;
 let SheetsDrawingRenderController = (_a9 = class extends Disposable {
-  constructor(_context, _sheetDrawingService, _lifecycleService) {
-    super(), this._context = _context, this._sheetDrawingService = _sheetDrawingService, this._lifecycleService = _lifecycleService, this._init();
+  constructor(_context, _sheetDrawingService, _drawingManagerService, _lifecycleService) {
+    super(), this._context = _context, this._sheetDrawingService = _sheetDrawingService, this._drawingManagerService = _drawingManagerService, this._lifecycleService = _lifecycleService, this._init();
   }
   _init() {
     this._drawingInitializeListener();
   }
   _drawingInitializeListener() {
     this._lifecycleService.lifecycle$.pipe(filter((e) => e === LifecycleStages.Steady), first()).subscribe(() => {
-      this._context.type === UniverInstanceType.UNIVER_SHEET && this._sheetDrawingService.initializeNotification(this._context.unitId);
+      this._context.type === UniverInstanceType.UNIVER_SHEET && (this._sheetDrawingService.initializeNotification(this._context.unitId), this._drawingManagerService.initializeNotification(this._context.unitId));
     });
   }
 }, __name(_a9, "SheetsDrawingRenderController"), _a9);
 SheetsDrawingRenderController = __decorateClass$1([
   __decorateParam$1(1, ISheetDrawingService),
-  __decorateParam$1(2, Inject(LifecycleService))
+  __decorateParam$1(2, IDrawingManagerService),
+  __decorateParam$1(3, Inject(LifecycleService))
 ], SheetsDrawingRenderController);
 var __defProp2 = Object.defineProperty, __getOwnPropDesc = Object.getOwnPropertyDescriptor, __defNormalProp2 = /* @__PURE__ */ __name((obj, key, value) => key in obj ? __defProp2(obj, key, { enumerable: !0, configurable: !0, writable: !0, value }) : obj[key] = value, "__defNormalProp"), __decorateClass = /* @__PURE__ */ __name((decorators, target, key, kind) => {
   for (var result = kind > 1 ? void 0 : kind ? __getOwnPropDesc(target, key) : target, i = decorators.length - 1, decorator; i >= 0; i--)

package/lib/types/controllers/render-controllers/sheet-drawing.render-controller.d.ts

@@ -1,11 +1,13 @@
 import { Disposable, LifecycleService } from '@univerjs/core';
+import { IDrawingManagerService } from '@univerjs/drawing';
 import { IRenderContext, IRenderModule } from '@univerjs/engine-render';
 import { ISheetDrawingService } from '@univerjs/sheets-drawing';
 export declare class SheetsDrawingRenderController extends Disposable implements IRenderModule {
     private _context;
     private readonly _sheetDrawingService;
+    private readonly _drawingManagerService;
     private _lifecycleService;
-    constructor(_context: IRenderContext, _sheetDrawingService: ISheetDrawingService, _lifecycleService: LifecycleService);
+    constructor(_context: IRenderContext, _sheetDrawingService: ISheetDrawingService, _drawingManagerService: IDrawingManagerService, _lifecycleService: LifecycleService);
     private _init;
     private _drawingInitializeListener;
 }

package/lib/types/controllers/sheet-drawing-update.controller.d.ts

@@ -30,6 +30,4 @@ export declare class SheetDrawingUpdateController extends Disposable implements
     private _updateImageListener;
     private _groupDrawingListener;
     private _focusDrawingListener;
-    private _drawingAddListener;
-    private _registerDrawing;
 }

package/lib/types/services/canvas-float-dom-manager.service.d.ts

@@ -4,10 +4,22 @@ import { IBoundRectNoAngle, Scene, SpreadsheetSkeleton, IRenderManagerService }
 import { ISheetDrawingService } from '@univerjs/sheets-drawing';
 import { CanvasFloatDomService } from '@univerjs/ui';
 export interface ICanvasFloatDom {
+    /**
+     * whether allow transform float-dom
+     */
     allowTransform: boolean;
+    /**
+     * initial position of float-dom
+     */
     initPosition: IPosition;
     componentKey: string;
+    /**
+     * unitId of workbook, if not set, will use current workbook
+     */
     unitId?: string;
+    /**
+     * subUnitId of worksheet, if not set, will use current worksheet
+     */
     subUnitId?: string;
     /**
      * @deprecated Please use `data`. for saving to disk, everything add to float-dom must be serializable.