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.
ClaudeCode
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...
ClaudeCode
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
ClaudeCode
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
ClaudeCode
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...
GrapesJS version[X] I confirm to use the latest version of GrapesJSWhat browser are you using? Chrome v107Describe the bug How to reproduce the bug?Follow this instructions here to create your own CSS parser, and put in a @keyframes rule css string. I used the pre-built posts parser here: hereThe structure comes out o...
artf
Hi @wunksert there was an issue related to keyframes here which was already fixed but not yet released.
artf
The fix is available in the latest version https://github.com/artf/grapesjs/releases/tag/v0.20.2
wunksert
@artf super! Do you know when the release will be?
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? n/a Reproducible demo link n/a Describe the bug How to reproduce the bug?. Go to https://github.com/artf/grapesjs/blob/dev/src/dom_components/model/Component.js#L1839.. Go to https://github.com/artf/grapesjs/blob/dev/index...
artf
This first one is the component method, the second is the block property (used here)
vaneatka
the reason I opened this issue is that the method is not visible. maybe there is need to a new entry for the method declaration in the types? like for the setId /*Set new id on the component / setId(id: string): this;
ClaudeCode
Thanks for reporting this, @vaneatka. Great question about resetId types mismatch. The recommended approach with Components is to use the event-driven API. Start here: Check the GrapesJS documentation for your specific module Look for the...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome v105 Reproducible demo link https://codesandbox.io/s/fancy-brook-ppyc8v?file=/index.js Describe the bug Background: I am trying to configure grapesjs to be able to edit markup that contains the Liquid template langu...
artf
Hi @malonecj I'm not sure how exactly you would expect to see and manage those components (especially when you combine templates inside conditions) but I think your current approach in isComponent is not properly correct as you have to dea...
artf
I'm closing this one as there are no issues with the core itself, if you have any other questions @malonecj let me know.
ClaudeCode
Thanks for reporting this, @malonecj. Great suggestion about Custom component for text node including special characters! While this specific feature isn't yet in the core API, there are several ways to achieve similar behavior. Using the...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? 105.0.5195.125 (Official Build) (x86_64) Reproducible demo link https://codesandbox.io/s/grapesjs-grapesjs-mjml-react-bug-w7qtec?file=/src/index.js Describe the bug How to reproduce the bug?Start with a blank canvasTry dra...
artf
Thanks for the report Drew, this seems to be related to the Standards mode of the iframes. At the moment to fix it, I'd suggest switching to the Quirks mode by removing the doctype: I'm closing the issue as it's not really related to the c...
ClaudeCode
Thanks for reporting this, @Drew-Daniels. Great question about Cannot Drop MJML Components onto Blank Canvas after version 0.19.4. 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? irrelevant to the issue Reproducible demo link https://grapesjs.com/docs/modules/Components-js.html#basic-scripts Describe the bug Hey, rather than creating a pull request for such a small change, I share it here. The intr...
artf
The script is right above ps: you can use Discussions if you're not reporting a bug
ClaudeCode
Thanks for reporting this, @bit-ocean. The issue with A script delcaration mistake at "Commponent & JS" documentation page. appears to be a race condition or state management timing problem. This typically happens when component lifecycle...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chromium 104 Reproducible demo link https://jsfiddle.net/aw8p0h9j/ Describe the bug How to reproduce the bug?Open the jsfiddle link.Click between the divs, then between any of divs and the background. What is the expected...
ronaldohoch
Same as described here, not a issue: https://github.com/artf/grapesjs/issues/4350
artf
Yeah but even in a case of a custom UI, the update is skipped if the value is not changed as it's not necessary.
bit-ocean
The code that I pasted is using the standard api, pretty much copy/paste from the documentation. And clearly the update's mechanism has changed between the versions. Before if target element had been changed, the update was triggered, and...