GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome v91 Reproducible demo link https://codepen.io/dhanadsp1120/pen/ExrRvOB Describe the bug How to reproduce the bug? 1.Disable the undo manager by invoking um.stop(). 2.Make alterations to the content. 3.Re-enable the...
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...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome v91 Reproducible demo link -- Describe the bug How to reproduce the bug? I wanted to add an change to the undomanager stack . var components = editor.Components; editor.UndoManager.add(components.getComponents().mod...
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...
GrapesJS version [x] I confirm to use the latest version of GrapesJS What browser are you using? Chrome Version 120.0.6099.225 Reproducible demo link https://jsfiddle.net/wattnu/y8L6whun/ Describe the bug How to reproduce the bug?Goto jsfiddlerun the codeon the canvas, select the blue div componenttry to drag the resi...
artf
Good catch, will fix it
GJS Helper
The issue you're encountering where configuring an onStart callback for the resizable property prevents the component from being resized is due to how GrapesJS (and often underlying drag/resize libraries) handles custom event callbacks. Wh...
GrapesJS version[X] I confirm to use the latest version of GrapesJSWhat browser are you using? Edge last versionReproducible demo link https://grapesjs.com/demoDescribe the bug How to reproduce the bug?Get an empty elementAdd double border with high value. (20..30px...)Move any block into it.What is the expected behav...
ronaldohoch
https://github.com/GrapesJS/grapesjs/pull/5622
GJS Helper
You've accurately identified a visual bug in GrapesJS related to how the placeholder for empty elements is positioned when the element has a significant border. This behavior is reproducible in the GrapesJS demo and affects the user experi...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Edge v120.0.2210.121 Reproducible demo link https://codepen.io/ashwinvalento/pen/jOJMPqx Describe the bug If we have set config.parser.optionsHtml.allowScripts: true and create a component with a script, Then when we impor...
artf
Reimporting directly JS as a component was never possible, so it's treated as a new one.
Ashwinvalento
Is there any way we can identify scripts of existing components and delete them on import? we wont be able to remove all the script tags since we would need to retain the scripts added manually.
GJS Helper
The issue you've encountered with script duplication when importing HTML multiple times in GrapesJS, especially with config.parser.optionsHtml.allowScripts: true, is a known behavior related to how GrapesJS reconciles components during set...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome Reproducible demo link None Describe the bug How to reproduce the bug?As per the definition for sectors in https://github.com/GrapesJS/grapesjs/blob/5c890449a52f526bc480d6dc649ff5e01ec59140/src/stylemanager/index.ts...
GJS Helper
You've accurately identified a type definition inconsistency in GrapesJS. The issue stems from a mismatch between the declared type for StyleManagerConfig.sectors.properties in src/stylemanager/config/config.ts and its actual usage and int...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome v120.0.6099.199 Reproducible demo link https://grapesjs.com/demo.html Describe the bug How to reproduce the bug?Drag the column 1 block onto the canvasDrag any other block into the cell for example textSelect the ce...
artf
Good catch, I'll fix it
GJS Helper
This is a known bug in GrapesJS related to how the Layers panel (specifically the Layer view) reacts to changes in a component's children collection. The issue arises when a component, like a column or cell, is initially empty and then has...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? MS Edge 120.0.2210.91 Reproducible demo link none available Describe the bug How to reproduce the bug?update to latest grapesjs versionset up a multi-page projectnote the current layer manager hierarchyinvoke the pagemanag...
artf
Thanks for the report @thigh will fix it
sunnynk19
Hi @thigh, Looks like still not working. Have you deployed the latest npm package?
artf
Not yet published, you can find here all the releases https://github.com/GrapesJS/grapesjs/releases
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome v91 Reproducible demo link https://grapesjs.com/demo.html Describe the bug How to reproduce the bug? Recently I have to upgrade grapesjs v0.14.62 to the latest version (0.21.8). In the old code, we used "storage:end...
artf
@vian013 please open bug issues only with a reproducible demo link, by a quick check the event seems to work properly
GJS Helper
The change in behavior for the storage:end:load event in GrapesJS from version 0.14.62 to 0.21.8 is intentional and reflects a more granular event system. In newer versions, storage:end:load is triggered specifically when the data has been...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Edge last version Reproducible demo link https://grapesjs.com/demo Describe the bug How to reproduce the bug?Open demo linkClick on HeaderChange view to mobile or tablet.Try to remove background from header. What is the ex...
artf
not a bug but a missing feature, we'll need to extend the stack type with a new configuration for "empty" values. I'll try to bring it in the next version.
artf
the clear icon from de property Background removes completely the properties from the selected device making it inherit again the values from the desktop settings. Correctthe clear icon in the layer line resets the stack values, setting th...
gustavohleal
I've inserted a solution for the stack type. I'm going to make a PR but I would like to ask you something first. The behavior i'm adding is:the clear icon in the layer line resets the stack values, setting the properties for the selected d...