fix: ensure the pointer init class is removed when updating locked st…

…atus
GrapesJS · Nov 18, 2024 · 66e38d8 · 66e38d8
1 parent 36572c3
commit 66e38d8
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 1 deletion.
diff --git a/packages/core/src/dom_components/view/ComponentView.ts b/packages/core/src/dom_components/view/ComponentView.ts
@@ -241,7 +241,7 @@ TComp> {
     const hoveredCls = `${ppfx}hovered`;
     const noPointerCls = `${ppfx}no-pointer`;
     const pointerInitCls = `${ppfx}pointer-init`;
-    const toRemove = [selectedCls, selectedParentCls, freezedCls, hoveredCls, noPointerCls];
+    const toRemove = [selectedCls, selectedParentCls, freezedCls, hoveredCls, noPointerCls, pointerInitCls];
     const selCls = extHl && !opts.noExtHl ? '' : selectedCls;
     this.$el.removeClass(toRemove.join(' '));
     const actualCls = el.getAttribute('class') || '';

diff --git a/packages/core/test/specs/dom_components/view/ComponentView.ts b/packages/core/test/specs/dom_components/view/ComponentView.ts
@@ -143,4 +143,20 @@ describe('ComponentView', () => {
     const result = model.getAttributes();
     expect(result.class).toEqual(undefined);
   });
+
+  test('updateStatus removes previous classes and adds new ones', () => {
+    model.addClass('selected');
+
+    model.set('locked', true);
+    view.updateStatus();
+    expect(view.el.getAttribute('class')).toEqual('no-pointer');
+
+    model.set('locked', false);
+    view.updateStatus();
+    expect(view.el.getAttribute('class')).toEqual('pointer-init');
+
+    model.set('locked');
+    view.updateStatus();
+    expect(view.el.getAttribute('class')).toEqual('');
+  });
 });