GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chromium v96 Reproducible demo link https://grapesjs.com/demo-newsletter-editor.html Describe the bug How to reproduce the bug?Open the developer tools at the console tabRun the code belowSelect the text addedEdit somethin...
shkhalid
+1
pety-dc
Hey I've been having similar issues. I think it's quite serious as both newsletter and mjml demo are affected. (Both use CKeditor as RTE) When I import a content that has a text (mj-text) that has styled elements (span) the imported conten...
raymondmakz
ckeditor output: <div>some rich text from <span style="color:blue">ckeditor</span></div> after parsing into editor, become like: <div data-gjs-type="text">some rich text from <span style="color:blue" data-gjs-type="default">ckeditor</span>...
GrapesJS version[x] I confirm to use the latest version of GrapesJSWhat browser are you using? Chrome 104.0.5112.79Reproducible demo link https://jsfiddle.net/gvrnw2c3/4/Describe the bug How to reproduce the bug? I'm currently unable to reproduce the bug using jsfiddle, which leads me to believe it's a problem with Ch...
tyuterry
A reproduce on Fiddle (with vue) I can not reproduce without vue jsfiddle Steps to reproduce the issue:Add a text componentSelect the text component then click the 'Convert to Conditional' button at the bottomHover over the conditional tex...
artf
Thanks guys, I was actually able to reproduce it by wrapping the example inside Vue but I'm not sure exactly why is stuck in the loop here. At first look, it seems to be related to Vue's Proxy observers. By comparing the demo with the prev...
benryanwilliams
Thanks @artf, all working perfectly now upon declaring the editor outside of the Vue instance.
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome Reproducible demo link https://grapesjs.com/demo.html Describe the bug What is the expected behavior? Button component is not working.Code of Conduct [X] I agree to follow this project's Code of Conduct
GJSBlock
Thanks for reporting this, @TechyMS. The issue with Button component is not working appears to be a race condition or state management timing problem. This typically happens when component lifecycle events and DOM modifications overlap, cr...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome 102.0.5005.115 (Official Build) (64-bit) Reproducible demo link Describe the bug This is an extension of #4473 When adding a map or video block via editor.setComponents() in a headless environment you get a similar...
artf
Thanks @skru I've actually already taken care of the parseUri here so it should work in the next release
GJSBlock
Thanks for reporting this, @skru. The issue with v0.19.5 Breaks headless editor.setComponents() (with Map/Video block) appears to be a race condition or state management timing problem. This typically happens when component lifecycle event...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Firefox v102.0.1 Reproducible demo link This is work code i can't.. Describe the bug How to reproduce the bug?Drag and drop on the page the block VideoYou'll see that you can't do anything with my iframe What is the expect...
artf
Thanks @ValouZ for the report, there is actually a typo on the iframe view. I'll take care of it for the next release.
GJSBlock
Thanks for reporting this, @ValouZ. Great question about Iframe on a block can't be edited. The recommended approach with Canvas is to use the event-driven API. Start here: Check the GrapesJS documentation for your specific module Look for...
GrapesJS version[X] I confirm to use the latest version of GrapesJSWhat browser are you using? Chrome 103.0.5060.134Reproducible demo link https://stackblitz.com/edit/node-zrdtg4?file=index.jsDescribe the bug How to reproduce the bug?Initialize Editor in headless modeLoad the project data with just a basic text node <...
skru
see #4473
artf
Fixed here https://github.com/artf/grapesjs/commit/f61222ec904ed56794f7b97e362ea2006dfceaf6
GJSBlock
Thanks for reporting this, @bimsina. Great question about Running in headless mode requires window. The recommended approach with StyleManager is to use the event-driven API. Start here: Check the GrapesJS documentation for your specific m...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome 102.0.5005.115 (Official Build) (64-bit) Reproducible demo link Describe the bug How to reproduce the bug? I'm unable to reproduce this on JSFiddle, but that sort of makes sense. #4373 has introduced a bug in editor...
artf
Thanks @skru I'll take care of it
GJSBlock
Thanks for reporting this, @skru. Great question about v0.19.5 Breaks headless editor.setComponents(): "Uncaught (in promise) ReferenceError: window is not defined". The recommended approach with Components is to use the event-driven API....
GrapesJS version [x] I confirm to use the latest version of GrapesJS What browser are you using? Chrome Reproducible demo link https://jsfiddle.net/weellingtonc/t8hs9f67/2/ Describe the bug How to reproduce the bug?add 'button-cube' component to workspaceclick on the created buttongo to the settings component and chan...
skru
first, you're code needs to go in a plugin https://grapesjs.com/docs/modules/Components.html#define-custom-component-type and you're not updating the model of the component, just the html in the canvas... https://jsfiddle.net/jmweg8u1/4/
artf
Yeah correct, thanks @skru You can also connect the content directly to the trait to read also the initial value.
GJSBlock
Thanks for reporting this, @capoia. Great question about Change in traits does not update html. The recommended approach with Components is to use the event-driven API. Start here: Check the GrapesJS documentation for your specific module...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome Version 103.0.5060.114 Reproducible demo link https://codepen.io/Dominic_M/pen/eYMgqyw Describe the bug How to reproduce the bug?Follow GrapesJS "Getting Started" >> https://grapesjs.com/docs/getting-started.htmlCon...
artf
@mickeyDominic you're placing all the default panels under the same object Where it should be one object per panel
mickeyDominic
Thank you @artf. Silly me :)
GJSBlock
Thanks for reporting this, @mickeyDominic. Great question about 'panel-devices' and 'panel-switcher' conflicting. The recommended approach with ProseMirror is to use the event-driven API. Start here: Check the GrapesJS documentation for yo...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? 103.0.5060.114 Reproducible demo link https://grapesjs.com/demo.html Describe the bug How to reproduce the bug?Click a parent container, such as the first component with the .container-width class in the demoIn typography,...
artf
Hi Barry, yeah currently this is the expected behavior (check also this discussion) but I hope one day to add proper support for inherited properties.
barryross
okay, thank you for letting me know On Mon, Jul 18, 2022 at 9:29 AM Artur Arseniev @.> wrote: Hi Barry, yeah currently this is the expected behavior (check also this discussion <https://github.com/artf/grapesjs/discussions/4245>) but I hop...
GJSBlock
Thanks for reporting this, @barryross. Great question about Style manager does not accurately reflect inherited color. The recommended approach with StyleManager is to use the event-driven API. Start here: Check the GrapesJS documentation...