#5618January 15, 2024by Ashwinvalento2 answers
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.
#5613January 12, 2024by SachinPuthiyothNo answers
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...
#5607January 10, 2024by Ju99ernaut1 answer
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
#5593January 3, 2024by thigh3 answers
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
#5584December 27, 2023by vian0131 answer
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
#5583December 26, 2023by ronaldohoch3 answers
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...
#5570December 20, 2023by prateekkumar-ec2 answers
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome Latest Version Reproducible demo link N/A( I can share video if required for further conversation) Describe the bug How to reproduce the bug?Suppose you have some content on your current page with some styles over t...
afrinxnahar
It would be great if you could kindly attach a video. Not sure about the empty pop-up thing
artf
@prateekkumar-ec this is not core related issue.
#5563December 18, 2023by leo-budgetsimpleNo answers
GrapesJS version[X] I confirm to use the latest version of GrapesJSWhat browser are you using? latest version of ChromeReproducible demo link https://grapesjs.com/demo.htmlDescribe the bug How to reproduce the bug? Taking an html string and rendering it in grapesjs with this code (I have allowScripts enabled) This is...
#5556December 13, 2023by pittleCheung1 answer
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? latest version of Chrome Reproducible demo link https://grapesjs.com/demo.html Describe the bug How to reproduce the bug? editor.addComponents( <ul class="mb-5 flex list-none flex-row flex-wrap border-b-0 pl-0" role="tabli...
artf
Already reported and fixed https://github.com/GrapesJS/grapesjs/issues/5525
#5549December 6, 2023by driskell3 answers
GrapesJS version[X] I confirm to use the latest version of GrapesJSWhat browser are you using? Safari 17Reproducible demo link https://grapesjs.com/demo-mjml.htmlDescribe the bug How to reproduce the bug?https://grapesjs.com/demo-mjml.htmlGo to view code - copy the MJMLGo to import code - paste in the MJML and add to...
artf
This is not core related issue and probably not an issue at all as <mj-preview> is not supported by default in mjml plugin
driskell
@artf Sorry I don't understand. It’s an issue as detailed. Is it something to raise on the other repository? I’m not familiar with the internals here so don’t know what it is that the GrapesJS/mjml needs to support in order for this reposi...
driskell
It is noted in Head: https://github.com/GrapesJS/mjml/blob/master/src/components/Head.ts#L14