@sequent-org/moodboard 1.2.14 → 1.2.16

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sequent-org/moodboard",
-  "version": "1.2.14",
+  "version": "1.2.16",
   "type": "module",
   "description": "Interactive moodboard",
   "main": "./src/index.js",

package/src/core/PixiEngine.js

@@ -170,12 +170,51 @@ export class PixiEngine {
     removeObject(objectId) {
         const pixiObject = this.objects.get(objectId);
         if (pixiObject) {
+            console.log('🗑️ PixiEngine: удаляем объект из сцены:', objectId);
+            
+            // Удаляем из родительского контейнера
             if (this.worldLayer) {
                 this.worldLayer.removeChild(pixiObject);
             } else {
                 this.app.stage.removeChild(pixiObject);
             }
+            
+            // ИСПРАВЛЕНИЕ: Полная очистка для изображений/эмоджи
+            if (pixiObject instanceof PIXI.Sprite) {
+                console.log('🗑️ PixiEngine: очищаем ресурсы изображения/эмоджи');
+                
+                // Очищаем текстуру (особенно важно для data URL)
+                if (pixiObject.texture && pixiObject.texture !== PIXI.Texture.WHITE) {
+                    // Не уничтожаем базовые текстуры PIXI
+                    const textureSource = pixiObject.texture.baseTexture?.resource?.src;
+                    if (textureSource && (textureSource.startsWith('data:') || textureSource.includes('emodji'))) {
+                        pixiObject.texture.destroy(false); // Уничтожаем только созданную текстуру
+                    }
+                }
+                
+                // Очищаем все события
+                pixiObject.removeAllListeners();
+                
+                // Принудительно уничтожаем спрайт
+                pixiObject.destroy({ children: true, texture: false, baseTexture: false });
+            } else {
+                // Для других типов объектов - стандартная очистка
+                if (pixiObject.destroy) {
+                    pixiObject.destroy({ children: true });
+                }
+            }
+            
+            // Удаляем из карты объектов
             this.objects.delete(objectId);
+            
+            // ПРИНУДИТЕЛЬНЫЙ РЕНДЕР после удаления
+            if (this.app && this.app.renderer) {
+                this.app.renderer.render(this.app.stage);
+            }
+            
+            console.log(`✅ PixiEngine: объект ${objectId} полностью удален и рендер обновлен`);
+        } else {
+            console.warn(`⚠️ PixiEngine: объект ${objectId} не найден для удаления`);
         }
     }
 

package/src/core/commands/DeleteObjectCommand.js

@@ -54,10 +54,14 @@ export class DeleteObjectCommand extends BaseCommand {
     }
 
     async execute() {
+        console.log('🗑️ DeleteObjectCommand: начинаем удаление объекта:', this.objectId);
+        
         // Удаляем объект из состояния и PIXI
         this.coreMoodboard.state.removeObject(this.objectId);
         this.coreMoodboard.pixi.removeObject(this.objectId);
         
+        console.log('🗑️ DeleteObjectCommand: объект удален из state и PIXI');
+        
         // Если это файловый объект с fileId, удаляем файл с сервера
         if (this.fileIdToDelete && this.coreMoodboard.fileUploadService) {
             try {
@@ -70,9 +74,12 @@ export class DeleteObjectCommand extends BaseCommand {
             }
         }
         
+        // Эмитим событие удаления для обновления всех UI компонентов
         this.coreMoodboard.eventBus.emit(Events.Object.Deleted, { 
             objectId: this.objectId 
         });
+        
+        console.log('✅ DeleteObjectCommand: событие Events.Object.Deleted отправлено');
     }
 
     undo() {

package/src/tools/object-tools/SelectTool.js

@@ -135,13 +135,28 @@ export class SelectTool extends BaseTool {
             // Обработка удаления объектов (undo создания, delete команды и т.д.)
             this.eventBus.on(Events.Object.Deleted, (data) => {
                 const objectId = data?.objectId || data;
-                if (objectId && this.selection.has(objectId)) {
+                console.log('🗑️ SelectTool: получено событие удаления объекта:', objectId, 'данные:', data);
+                
+                // ЗАЩИТА: Проверяем что данные валидны
+                if (!objectId) {
+                    console.warn('⚠️ SelectTool: получено событие удаления с невалидным objectId');
+                    return;
+                }
+                
+                if (this.selection.has(objectId)) {
+                    console.log('🗑️ SelectTool: удаляем объект из selection:', objectId);
                     this.removeFromSelection(objectId);
                     
-                    // Если выделение стало пустым, скрываем ручки
+                    // ИСПРАВЛЕНИЕ: Принудительно очищаем selection если он стал пустым
                     if (this.selection.size() === 0) {
+                        console.log('🗑️ SelectTool: selection пустой, скрываем ручки');
+                        this.emit(Events.Tool.SelectionClear);
                         this.updateResizeHandles();
                     }
+                } else {
+                    console.log('🗑️ SelectTool: объект не был в selection, обновляем ручки на всякий случай');
+                    // Принудительно обновляем ручки без излишних действий
+                    this.updateResizeHandles();
                 }
             });
 		}

package/src/ui/CommentPopover.js

@@ -132,7 +132,8 @@ export class CommentPopover {
 
     _onDocMouseDown(e) {
         if (!this.popover || this.popover.style.display === 'none') return;
-        if (this.popover.contains(e.target)) return; // клик внутри окна — не закрываем
+        // ИСПРАВЛЕНИЕ: Защита от null элементов
+        if (this.popover && e.target && this.popover.contains(e.target)) return; // клик внутри окна — не закрываем
         this.hide();
     }
 

package/src/ui/ContextMenu.js

@@ -51,7 +51,8 @@ export class ContextMenu {
         // Скрывать при клике вне меню или по Esc
         document.addEventListener('mousedown', (e) => {
             if (!this.isVisible) return;
-            if (!this.element.contains(e.target)) {
+            // ИСПРАВЛЕНИЕ: Защита от null элементов
+            if (this.element && e.target && !this.element.contains(e.target)) {
                 this.hide();
             }
         });

package/src/ui/FramePropertiesPanel.js

@@ -435,7 +435,8 @@ export class FramePropertiesPanel {
     }
 
     _documentClickHandler(e) {
-        if (this.colorPalette && !this.colorPalette.contains(e.target) && 
+        // ИСПРАВЛЕНИЕ: Защита от null элементов
+        if (this.colorPalette && e.target && !this.colorPalette.contains(e.target) && 
             this.colorButton && !this.colorButton.contains(e.target)) {
             this._hideColorPalette();
         }

package/src/ui/HtmlHandlesLayer.js

@@ -35,6 +35,23 @@ export class HtmlHandlesLayer {
         this.eventBus.on(Events.Tool.SelectionRemove, () => this.update());
         this.eventBus.on(Events.Tool.SelectionClear, () => this.hide());
         this.eventBus.on(Events.Tool.DragUpdate, () => this.update());
+        
+        // ИСПРАВЛЕНИЕ: Обработка удаления объектов
+        this.eventBus.on(Events.Object.Deleted, (data) => {
+            const objectId = data?.objectId || data;
+            console.log('🗑️ HtmlHandlesLayer: получено событие удаления:', data, 'objectId:', objectId);
+            
+            // Принудительно скрываем и очищаем все ручки
+            this.hide();
+            
+            // Очищаем DOM от старых ручек 
+            this.layer.innerHTML = '';
+            
+            // Обновляем для актуального состояния
+            setTimeout(() => {
+                this.update();
+            }, 10); // Небольшая задержка для полной очистки
+        });
         this.eventBus.on(Events.Tool.DragStart, () => { this._handlesSuppressed = true; this._setHandlesVisibility(false); });
         this.eventBus.on(Events.Tool.DragEnd, () => { this._handlesSuppressed = false; this._setHandlesVisibility(true); });
         this.eventBus.on(Events.Tool.ResizeUpdate, () => this.update());

package/src/ui/MapPanel.js

@@ -38,6 +38,8 @@ export class MapPanel {
         // Закрытие по клику вне панели
         document.addEventListener('mousedown', (e) => {
             if (!this.popupEl) return;
+            // ИСПРАВЛЕНИЕ: Защита от null элементов
+            if (!this.element || !e.target) return;
             if (this.element.contains(e.target)) return;
             this.hidePopup();
         });
@@ -45,7 +47,8 @@ export class MapPanel {
         // Колесо для зума внутри миникарты
         this.element.addEventListener('wheel', (e) => {
             if (!this.popupEl) return;
-            if (!this.popupEl.contains(e.target)) return;
+            // ИСПРАВЛЕНИЕ: Защита от null элементов
+            if (!this.popupEl || !e.target || !this.popupEl.contains(e.target)) return;
             e.preventDefault();
             // Масштабируем вокруг точки под курсором в миникарте
             const rect = this.canvas.getBoundingClientRect();

package/src/ui/NotePropertiesPanel.js

@@ -448,14 +448,16 @@ export class NotePropertiesPanel {
         let shouldClose = true;
         
         for (let palette of palettes) {
-            if (palette && palette.contains(e.target)) {
+            // ИСПРАВЛЕНИЕ: Защита от null элементов
+            if (palette && e.target && palette.contains(e.target)) {
                 shouldClose = false;
                 break;
             }
         }
         
         for (let button of buttons) {
-            if (button && button.contains(e.target)) {
+            // ИСПРАВЛЕНИЕ: Защита от null элементов
+            if (button && e.target && button.contains(e.target)) {
                 shouldClose = false;
                 break;
             }

package/src/ui/TextPropertiesPanel.js

@@ -259,7 +259,8 @@ export class TextPropertiesPanel {
 
         // Закрываем панель при клике вне её
         document.addEventListener('click', (e) => {
-            if (!colorSelectorContainer.contains(e.target)) {
+            // ИСПРАВЛЕНИЕ: Защита от null элементов
+            if (!colorSelectorContainer || !e.target || !colorSelectorContainer.contains(e.target)) {
                 this._hideColorDropdown();
             }
         });
@@ -460,7 +461,8 @@ export class TextPropertiesPanel {
 
         // Закрываем панель при клике вне её
         document.addEventListener('click', (e) => {
-            if (!bgSelectorContainer.contains(e.target)) {
+            // ИСПРАВЛЕНИЕ: Защита от null элементов
+            if (!bgSelectorContainer || !e.target || !bgSelectorContainer.contains(e.target)) {
                 this._hideBgColorDropdown();
             }
         });
@@ -871,7 +873,7 @@ export class TextPropertiesPanel {
     }
 
     _onDocMouseDown(e) {
-        // Скрываем панель при клике вне неё и вне текстового объекта
+        // ИСПРАВЛЕНИЕ: Защита от null элементов + скрываем панель при клике вне неё
         if (!this.panel || !e.target) return;
         
         // Если клик внутри панели - не скрываем

package/src/ui/Toolbar.js

@@ -555,7 +555,10 @@ export class Toolbar {
 
         // Клик вне попапов — закрыть
         document.addEventListener('click', (e) => {
-            const isInsideToolbar = this.element.contains(e.target);
+            // ИСПРАВЛЕНИЕ: Защита от null элементов
+            if (!e.target) return;
+            
+            const isInsideToolbar = this.element && this.element.contains(e.target);
             const isInsideShapesPopup = this.shapesPopupEl && this.shapesPopupEl.contains(e.target);
             const isInsideDrawPopup = this.drawPopupEl && this.drawPopupEl.contains(e.target);
             const isInsideEmojiPopup = this.emojiPopupEl && this.emojiPopupEl.contains(e.target);

package/src/ui/ZoomPanel.js

@@ -74,6 +74,8 @@ export class ZoomPanel {
 
         document.addEventListener('mousedown', (e) => {
             if (!this.menuEl) return;
+            // ИСПРАВЛЕНИЕ: Защита от null элементов
+            if (!this.element || !e.target) return;
             if (this.element.contains(e.target)) return;
             this.hideMenu();
         });