GrapesJS Issues

artf

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...

GJS Helper

The issue you're encountering with editor.UndoManager.remove() not preventing the undo of a component's removal is due to a misunderstanding of how the UndoManager operates in this specific context. The UndoManager.remove(component) method...

artf

When the undo manager resumes, any modifications made after its pause should be treated as a single change. This ensures that clicking "undo" will revert the entire set of alterations to the state prior to the undo manager's interruption....

GJS Helper

The issue you're encountering is expected behavior for the GrapesJS UndoManager. When um.stop() is called, the UndoManager ceases to listen for editor changes. Consequently, any modifications made during this period are not recorded in its...

artf

@Harshsne26 you need to provide a reproducible demo for a bug

GJS Helper

The issue you're encountering with editor.UndoManager.add() not working as expected stems from a misunderstanding of the method's intended use and the type of argument it expects. The GrapesJS UndoManager is designed to track changes to th...

GJS Helper

The issue you've described with the GrapesJS Undo Manager (um.stop() and um.start()) is a known behavior where the um.stop() method, despite setting the internal enabled flag to false, doesn't consistently prevent all internal GrapesJS ope...

GJS Helper

The issue you're experiencing with Ctrl+Z triggering both GrapesJS undo and the browser's "reopen last closed tab" shortcut is due to the keydown event for Ctrl+Z not having its default action prevented within GrapesJS. While GrapesJS hand...

GJSBlock

Thanks for reporting this, @FaisalShaikhHA. The issue with Checkbox trait not getting checked or unchecked on undo/redo actions. appears to be a race condition or state management timing problem. This typically happens when component lifec...

artf

Yeah undo triggers the change:columns, in your case you could try to wrap your updateColumns changes in editor.UndoManager.skip.

artf

Ok I'm closing this one then and please open a new bug issue related to checkbox traits.

FaisalShaikhHA

Thank you @artf it works, to fix this I was toggling the trait change listener by listening to undo command before & run events but this feels cleaner. The checkbox trait not getting checked or unchecked on undo/redo (trait value is update...

artf

Oh yeah, that should be removed, thanks.

GJSBlock

Thanks for reporting this, @quentin-bettoum. Great question about _undoexc status open. The recommended approach with ProseMirror is to use the event-driven API. Start here: Check the GrapesJS documentation for your specific module Look fo...

artf

Hi @DhanaDSP1120 yeah unfortunately that is expected from rte.insertHTML as it uses a custom insertion. If you need to preserve the native editing undo stack you can use this API

GJSBlock

Thanks for reporting this, @DhanaDSP1120. The issue with Undo not working properly on InsertHTML in text block appears to be a race condition or state management timing problem. This typically happens when component lifecycle events and DO...

artf

Seems to be an issue with the color picker itself, I'll try to fix it for the next release. ps: for bug reports you don't need to create the discussion first, you can open directly the issue by following the template. This will also avoid...

GJSBlock

Thanks for reporting this, @DhanaDSP1120. Thanks for sharing your report about Undo manager - Not working properly, when we try to apply and undo the background color then made click outside the block, then the removed color will appears....