GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? latest chrome, firefox, edge Reproducible demo link https://grapesjs.com/demo Describe the bug How to reproduce the bug?drag a acomponent in layer manager and try to put it between other items under bodynot allowed. nestin...
artf
Will be fixed in the next release. Thanks @sumasal for the report and @mohamedsalem401 for the fix ๐โโ๏ธ
sumasal
issue title is somehow clipped. what i meant is it seems to work on older versions such as 0.21.8 and below
GJS Helper
The GrapesJS Layer Manager is designed to allow both reordering components on the same level and nesting components within others. Based on the official GrapesJS demo (https://grapesjs.com/demo) and the latest version of GrapesJS, this fun...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Firefox 131.0 (64-bit) on Linux and Mac Reproducible demo link https://grapesjs.com/demo Describe the bug How to reproduce the bug?Start dragging the "Hover me" buttonTry to put it above the "All text blocks could be edite...
padcom
You guys are awesome! And not a moment too soon :) I have a presentation Today in the afternoon and now I'll be able to show it working on Firefox too!
padcom
I've tracked down the breaking change in Firefox to version 109.0. It's the first one that has the problem. Version 108.0.2 works fine. Hope that helps.
padcom
I have also tested 133.0a1 nightly from 2024-10-24-21-16-15-mozilla-central and the problem is also there.
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome Version 130.0.6723.59 (Official Build) (arm64) Reproducible demo link https://codepen.io/btmkt/pen/QWeOJRB?editors=1111 Describe the bug How to reproduce the bug?Open console.Drop multiple components inside the canv...
artf
@mohamedsalem401 this is probably happening due to the shallow editor. I think it would make sense to leverage the temporary option (which we already use to skip some actions) in canMove. wdyt?
mohamedsalem401
@artf Yes, I suspect it has to do with the shallow editor. Hopefully the temporary option will work and then I'll open a PR
bt-mkt
@mohamedsalem401 this is probably happening due to the shallow editor. I think it would make sense to leverage the temporary option (which we already use to skip some actions) in canMove. wdyt? Hello @artf, The change above, which I saw wa...
GrapesJS version[X] I confirm to use the latest version of GrapesJSWhat browser are you using? ChromeReproducible demo link https://codepen.io/Harshsne/pen/xxoOyXODescribe the bug How to reproduce the bug?add link to a word using rte, link component is added Now type something after the link component without space an...
sirbeagle
Alight - the only bad part is we may never know what the fix ended up being. @artf - I think we can close out this issue.
sirbeagle
So I've been playing around with this quite a bit and I can reproduce the error on the Codepen that @yashvi2026 provided. My steps to reproduce (most of the time) are very similar: Type two words with numerous spaces in between in a text b...
yashvi2026
Yes.After upgrading to latest version ,this issue doesnt occur.
GrapesJS version[X] I confirm to use the latest version of GrapesJSWhat browser are you using? Microsoft Edge 130.0.2849.46Reproducible demo link https://jsfiddle.net/uwybfd98/Describe the bug How to reproduce the bug?select the carousel componentclick the space in the upper right or lower left corner of the image and...
artf
@hr1201 onRender could be triggered many times, you should never attach listeners there if you don't clean up the properly. Your logic is also purely model related so it shouldn't even be in the view, just move your code in the model.init...
GJS Helper
The issue you're experiencing with recursive component:clone events and editor jams stems from how event listeners are attached within your custom component's view.onRender method. Let's break down the root cause and provide a robust solut...
GrapesJS version[X] I confirm to use the latest version of GrapesJSWhat browser are you using? Chrome 129.0.6668.101Reproducible demo link https://grapesjs.com/demoDescribe the bug How to reproduce the bug? open https://grapesjs.com/demo run this script in DevTools: What is the expected behavior? Successfully copied w...
maxming2333
In addition to the above method, is there any other way to copy the page?
GJS Helper
The TypeError: Converting circular structure to JSON you're encountering when cloning a GrapesJS page's main component and adding it to a new page is a common issue related to how GrapesJS components are serialized for storage. This happen...
GrapesJS version[X] I confirm to use the latest version of GrapesJSWhat browser are you using? ChromeReproducible demo link ---Describe the bug editor.getModel().initBaseColorPicker(DomElement, { / spectrum options / }); after initailization when the palete opens i want to set a color to it ? @artf Code of Conduct[X]...
NeonLexie
This looks interesting. I wouldn't mind taking a crack at it.
yashvi2026
GJS Helper
To set an initial color for a custom color picker initialized with editor.getModel().initBaseColorPicker(), you need to pass the color option within the Spectrum.js options object. GrapesJS's initBaseColorPicker method internally uses Spec...
GrapesJS version [x] I confirm to use the latest version of GrapesJS What browser are you using? Chrome Reproducible demo link https://jsfiddle.net/a2hbxg0y/1/ Describe the bug How to reproduce the bug?Open dev tools and write element style for iframe body - zoom: n; What is the current behavior? When hovering over an...
artf
You shouldn't edit the zoom of the iframe, use the native API
GJS Helper
You've identified a specific bug in GrapesJS related to the showOffsets feature when the iframe body has a CSS zoom property applied. This is a valid observation, and I can confirm the behavior you described. Issue Confirmation When zoom i...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Firefox 131.0.2 Reproducible demo link https://grapesjs.com/demo-newsletter-editor.html Describe the bug How to reproduce the bug?Go to the grapesjs demo, then import the following html:Open your browser console.Rapidly cl...
artf
Hey @jlafosse thanks for the report. I'd not expect that issue with the latest version: <img width="123" alt="Screenshot 2024-10-21 at 19 00 04" src="https://github.com/user-attachments/assets/ff074a31-cd19-4547-bb27-596c92e16d64"> Can you...
jlafosse
Indeed, that appears to have fixed the issue! Thank you!
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? 129 Reproducible demo link https://grapesjs.com/demo Describe the bug const newEditor = grapesjs.init({ fromElement: true, // Allow components from HTML container: "#editor", canvas: { scripts: [ "https://cdn.jsdelivr.net/...
artf
@klipto-inc please create the bug report with a proper reproducible live demo
GJS Helper
The issue you're encountering with Preline.js not working on the GrapesJS canvas, despite being included via canvas.scripts, is a common timing problem with UI libraries that rely on DOMContentLoaded for initialization. Root Cause Many Jav...