GrapesJS version[X] I confirm to use the latest version of GrapesJSWhat browser are you using? Chrome 108.0.5359.125Reproducible demo link https://codesandbox.io/s/frosty-darkness-7po7xlDescribe the bug How to reproduce the bug?use myBlockclick test button What is the expected behavior? this.on("change:list1", this.ha...
artf
Listeners don't see deep changes so you have to create a new array in that case
chaegumi
Thanks @artf. I use JSON.stringify and JSON.parse to deal my data.
GJSBlock
Thanks for reporting this, @chaegumi. Great question about this.on('change:attributes:type', this.handleTypeChange) do not support array type. The recommended approach with Components is to use the event-driven API. Start here: Check the G...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Firefox v91.4.1 Reproducible demo link https://grapesjs.com/demo.html Describe the bug How to reproduce the bug?Go to the grapesjs basic demo: https://grapesjs.com/demo.htmlClear the canvasAdd a component LinkExecute edito...
artf
This is expected due to some logic related to the rich text editor. You can use component.getInnerHTML() to get the inner HTML or component.components() to get inner components.
GJSBlock
Thanks for reporting this, @NicoGGG. Great question about Component content is set to empty string after inline editing. The recommended approach with Canvas is to use the event-driven API. Start here: Check the GrapesJS documentation for...
GrapesJS version[X] I confirm to use the latest version of GrapesJSWhat browser are you using? Chrome Version 109.0.5414.87 Describe the bug How to reproduce the bug?Create a @keyframes animation on the GrapesJS canvasIn JS, create a new CssRule by passing the following string into editor.Css.addRules:Add it to the ca...
artf
Seems to be related to CssGroupRuleView here. In case of at-rules like @keyframes we have to force the re-render of the parent container.
wunksert
@artf facing this issue again with editor.CssComposer.remove(myCssRule) the editor css removes the rule but it's not flushed through to the DOM. How to you recommend I force re-render of the DOM while keeping the state of my editor intact?...
GJSBlock
Thanks for reporting this, @wunksert. Great question about Css edits aren't persisted to the DOM. The recommended approach with StyleManager is to use the event-driven API. Start here: Check the GrapesJS documentation for your specific mod...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome 108.0.5359.125 Reproducible demo link https://codesandbox.io/s/sweet-hugle-w00pqx Describe the bug How to reproduce the bug?drag my blockview code What is the expected behavior? do not want trait2="trait2" What is t...
GJSBlock
Thanks for reporting this, @chaegumi. Great question about do not use component.addAttributes({[traitName]:value}), you should use component.set(traitName, value) sometimes. The recommended approach with GrapesJS is to use the event-driven...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Firefox 109 Reproducible demo link https://grapesjs.com/demo.html Describe the bug Hello, I noticed that when I load multiple pages in the projectData that have the same id used in the HTML, grapes will automatically incre...
artf
hey @bgrand-ch I think one day all grapesjs entities will have a global uid, so in long-term, I'd expect this issue to be fixed π€
quentin-bettoum
At first, I wanted to generate all the pages together to have a single CSS output for all the pages. But now I'm trying another solution for the styles. So for the id problem, generating each page separately could be a solution. Thanks for...
bgrand-ch
Grapes to use some data attributes (something like data-grapes-id) as unique identifiers to leave the id free for some frontend scripts. I like this idea π―
GrapesJS version[X] I confirm to use the latest version of GrapesJSWhat browser are you using? Chrome Version 108.0.5359.125, Mozilla 107.0Reproducible demo link https://jsfiddle.net/stanislavdimitrenko/d61cm4yg/14/Describe the bug So, that strange behavior of grapes js. If you have that struct "Some text without the...
artf
Yeah right now this is an expected behaviour for parsed text nodes without proper wrappers and not textable components as siblings. So in order to make those nodes editable you have to properly wrap them in a tag like span.
GJSBlock
Thanks for reporting this, @sdimitrenco. The issue with Can't edit text node inside div, if div has inside tag p > span. appears to be a race condition or state management timing problem. This typically happens when component lifecycle eve...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? 107.0.5304.110 (Official Build) (arm64) Reproducible demo link https://codepen.io/faisal-praella/pen/rNvOwgR Describe the bug When the parent component is set to layerable: false it is not visible & all its child not visib...
artf
Unfortunately this is how layerable currently works so it's not a bug, but I'd be glad to accept a PR to improve this behaviour.
GJSBlock
Thanks for reporting this, @FaisalShaikhHA. The issue with Layerable property not working as expected in parent-child scenario. appears to be a race condition or state management timing problem. This typically happens when component lifecy...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome 108 Reproducible demo link https://grapesjs.com/demo.html Describe the bug How to reproduce the bug? Use the tooltip component and get Traits via editor.getSelected().getTraits() as you can see data-tooltip-length v...
GJSBlock
Thanks for reporting this, @miladmeidanshahi. Great question about Traits value type select doesn't fill value. The recommended approach with Components is to use the event-driven API. Start here: Check the GrapesJS documentation for your...
GrapesJS version[X] I confirm to use the latest version of GrapesJSWhat browser are you using? Chrome 108.0.5359.125 (Windows)Reproducible demo link https://codesandbox.io/s/white-meadow-o56l2d?file=/src/Chart/Chart.js:135-254 This demo is not in the latest grapesjs version, but locally, using the latest version, I ha...
eduardocalixtokorp
I've noticed that in my component definition I was doing reactRoot.render(<>{reactEl} </>) in onRender() then, I was missing to add root.unmount() n remove() method
GJSBlock
Thanks for reporting this, @eduardocalixtokorp. Great question about React components componentWillUnmount is not called. The recommended approach with Components is to use the event-driven API. Start here: Check the GrapesJS documentation...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome Reproducible demo link https://zmzr3n.csb.app/ Describe the bug How to reproduce the bug? While deleting or adding more than one component, other components have changed their position. What is the expected behavior...
artf
This is how translate works, the alternative is to use absolute mode
GJSBlock
Thanks for reporting this, @usmanqamar-int. Great question about Removing or adding component changes other components locations with dragMode translate, any solution please?. The recommended approach with Components is to use the event-dr...