Issue #5641Opened January 24, 2024by yashvi20260 reactions

BUG: Undomanager.remove() not working

Question

GrapesJS version

  • I confirm to use the latest version of GrapesJS

What browser are you using?

Chrome v91

Reproducible demo link

Code SnippetTEXT
https://codepen.io/Harshsne/pen/WNgZzPE

### Describe the bug

**How to reproduce the bug?**
1. Append an image to the canvas and  close the image modal.(so that component will be removed from tracking as i have removed the component from the undomanager stack)
` editor.on('asset:close', () => {
   console.log("close")
    const selected = editor.getSelected();  
    if (selected && selected.is('image')) {
  editor.UndoManager.remove(selected);  
    }
  })`
2. Press crtl+ z

**What is the expected behavior?**
...Removed component from the undomanager should not be tracked and when i press crtl+z its changes should not be displayed.

**What is the current behavior?**
Removed component from undomanager is still tracked by undomanager.

![undomanager new](https://github.com/GrapesJS/grapesjs/assets/133509113/d1bc9720-161c-4839-8810-7bba8c9c7d7c)

My actual need is:
when i close the image modal without adding any image to it ,I have removed the image block as in my code  ,so when i press undo without any src the appended image block is displayed i dont want that to happen so i would like to remove that component from the undomanager stack .
` editor.on('asset:close', () => {
    const selected = editor.getSelected();
    if (selected && selected.is('image')) {
      if (selected.get('attributes').src == "empty") {
        editor.getSelected().remove();
        editor.select();
       editor.UndoManager.remove(selected);
    }
    }
  })`
Any other alternative way i can achieve this? @artf 
### Code of Conduct

- [X] I agree to follow this project's Code of Conduct

Answers (1)

artfFebruary 10, 20240 reactions

The problem here is not Undomanager.remove but what is tracked. When you're resizing the image, you're not editing the Component itself but its CssRule object so that's why it's not working as expected. I'd probably suggest to simply skip the undo manager with operations you don't want to track

editor.UndoManager.skip(() => {
  // ....
});

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.