Issue #4136Opened February 9, 2022by m-jojo-s1 reactions

BUG: component.remove() in 'component:mount' does not remove the rendered view

Question

GrapesJS version

  • I confirm to use the latest version of GrapesJS

What browser are you using?

Chrome Version 98.0.4758.80 (Official Build) (arm64)

Reproducible demo link

https://jsfiddle.net/Lsjhxczp/10/

Describe the bug

How to reproduce the bug?

  1. Setup the component:mount hook to remove mounted component
  2. Add a new block/component

What is the expected behavior? The new component gets removed, both model and view.

What is the current behavior? Only the model is removed, view stays in place and isn't re-rendered after removal.

If is necessary to execute some code in order to reproduce the bug, paste it here below:

editor.on('component:mount', component => {
    if (component.is('image')) {
      component.remove();
    }
});

// the following has no effect
/* editor.on('component:create', component => {
    if (component.is('image')) {
      component.remove();
    }
}); */

Code of Conduct

  • I agree to follow this project's Code of Conduct

Answers (1)

artfFebruary 11, 20221 reactions

Yeah, the problem here is the DOM node is created but not yet appended to the parent. You can defer the removal via setTimeout but that will still trigger the image activation. Probably I'll move that event once the node is appended.

Related Questions and Answers

Continue research with similar issue discussions.

Paid Plugins That Match This Issue

Curated by issue keywords and label relevance to help you ship faster.

View all plugins

Loading paid plugin recommendations...

Browse Plugin Categories

Jump directly to plugin category pages on the marketplace.