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...
GJS Helper
The behavior you're observing with script tag attributes not appearing on the GrapesJS canvas, while being present in the exported HTML, is expected and by design for how GrapesJS handles non-visual elements, especially script tags carryin...
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
GJS Helper
The issue you're experiencing, where GrapesJS crashes or becomes unresponsive when adding and selecting elements with a large number of utility classes (especially from frameworks like Tailwind CSS or component libraries using data-[...] a...
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
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Version 119.0.6045.200 Reproducible demo link https://jsfiddle.net/d64pkcom/1/ Describe the bug How to reproduce the bug?Drag component text in CanvasWatch the console of jsfiddle What is the expected behavior? Don't trigg...
GJS Helper
You've accurately identified a minor, yet unexpected, behavior in GrapesJS where the rte:disable event is triggered when dragging a new component (like a text block) from the blocks panel onto the canvas. This is indeed not the expected be...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome 119.0.6045.200 64 bits Reproducible demo link https://grapesjs.com/demo.html Describe the bug How to reproduce the bug?Drag and drop a video block into the canvasClick on the video and select YouTube as the Provider...
artf
Looks like the muted property was correct for Vimeo but not for Youtube. I'll push the fix, thanks.
GJS Helper
Thank you for reporting this issue and for your thorough investigation. You've correctly identified the root cause and a working solution. This is indeed a bug in GrapesJS's default YouTube video component. Root Cause Analysis The issue st...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome Version 119 Reproducible demo link https://jsfiddle.net/bj7afw63/1/ Describe the bug How to reproduce the bug?Load the fiddleTry to add a link with the default RTE (not CKE)Nothing happens in the UI and a JS error a...
GJSBlock
Thanks for reporting this, @adiux. Great question about Can not add a link: getModel() fails with: TypeError: $ is not a function. The recommended approach with ProseMirror is to use the event-driven API. Start here: Check the GrapesJS doc...
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?Disable the undo manager by invoking um.stop().Make alterations to the content.Re-enable the undo mana...
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...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Edge v119.0.2151.72 Reproducible demo link https://grapesjs.com/demo.html Describe the bug Hi, thank you for creating an amazing project. I would like to report an issue found during my use of GrapesJS. How to reproduce th...
artf
Agree with the suggested alternative, I will update accordingly. Thanks @duydvu
GJSBlock
Thanks for reporting this, @duydvu. Great question about Performance issue when adding more than 20 classes to a single component. The recommended approach with StyleManager is to use the event-driven API. Start here: Check the GrapesJS do...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Firefox Reproducible demo link https://grapesjs.com/demo.html Describe the bug How to reproduce the bug?Select a component on stageCopy the component with CTRL+cSelect the body (deselect the component)Paste with CTRL+v Wha...
lexoyo
Note that this bug makes it impossible to copy paste elements to an empty page
GJS Helper
The issue you've described is a known behavior in GrapesJS's default paste-component command. The root cause lies in the command's implementation, specifically the condition if (!selected || !selected.length) return; at the beginning of it...
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 replaceWith method from Component.ts doesn't have a type for string. What is the expected behavior? I should be able to replace a component with a string like in the exam...
You've correctly identified a TypeScript typing issue with the replaceWith method in GrapesJS. While the method internally supports replacing a component with an HTML string, its TypeScript definition (Component.d.ts) currently doesn't ref...