@sequent-org/moodboard 1.2.15 → 1.2.16

package/package.json

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

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

@@ -135,9 +135,15 @@ export class SelectTool extends BaseTool {
             // Обработка удаления объектов (undo создания, delete команды и т.д.)
             this.eventBus.on(Events.Object.Deleted, (data) => {
                 const objectId = data?.objectId || data;
-                console.log('🗑️ SelectTool: получено событие удаления объекта:', objectId);
+                console.log('🗑️ SelectTool: получено событие удаления объекта:', objectId, 'данные:', data);
                 
-                if (objectId && this.selection.has(objectId)) {
+                // ЗАЩИТА: Проверяем что данные валидны
+                if (!objectId) {
+                    console.warn('⚠️ SelectTool: получено событие удаления с невалидным objectId');
+                    return;
+                }
+                
+                if (this.selection.has(objectId)) {
                     console.log('🗑️ SelectTool: удаляем объект из selection:', objectId);
                     this.removeFromSelection(objectId);
                     
@@ -147,10 +153,9 @@ export class SelectTool extends BaseTool {
                         this.emit(Events.Tool.SelectionClear);
                         this.updateResizeHandles();
                     }
-                } else if (objectId) {
-                    console.log('🗑️ SelectTool: объект не был в selection, но проверяем ручки');
-                    // Дополнительная проверка - возможно объект был удален не через selection
-                    // Принудительно обновляем ручки
+                } 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

@@ -37,8 +37,9 @@ export class HtmlHandlesLayer {
         this.eventBus.on(Events.Tool.DragUpdate, () => this.update());
         
         // ИСПРАВЛЕНИЕ: Обработка удаления объектов
-        this.eventBus.on(Events.Object.Deleted, ({ objectId }) => {
-            console.log('🗑️ HtmlHandlesLayer: объект удален, принудительно очищаем ручки:', objectId);
+        this.eventBus.on(Events.Object.Deleted, (data) => {
+            const objectId = data?.objectId || data;
+            console.log('🗑️ HtmlHandlesLayer: получено событие удаления:', data, 'objectId:', objectId);
             
             // Принудительно скрываем и очищаем все ручки
             this.hide();

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();
         });