GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Opera 10.0.5130.23 | Linux Reproducible demo link https://grapesjs.com/demo.html Describe the bug Clear canvasFrom Blocks menu add 1 Column component -> Settings tab is opened automatically :ok_hand: Click newly added comp...
artf
Thanks @gladykov but this is not related to the core, it's just an example behavior implemented in the demo.
GJS Helper
This is a valid observation regarding user experience in GrapesJS. The behavior you're describing is not a bug in the sense of a broken feature, but rather a UX inconsistency where clicking an already selected component doesn't always re-a...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome V124 Reproducible demo link https://jsfiddle.net/ycvz2gno/7/ Describe the bug How to reproduce the bug?Put a text block in the canvasDouble click on the content inside the text blockThe content cannot be edited What...
Dobby85
By looking into the code, I saw that to get the RTE functionnality, our new component has to extend ComponentTextView. My initial issue was that I did something like : And it tooks the view of my baseStructure and not the text component. I...
GJS Helper
The issue you're encountering where text components loaded via fromElement: true are not editable, even with data-gjs-type="text" and editable="true" in the HTML, stems from how GrapesJS handles existing DOM elements during initialization....
GrapesJS version[X] I confirm to use the latest version of GrapesJSWhat browser are you using? Chrome v123Reproducible demo link https://jsfiddle.net/xwgons8z/Describe the bug I'm trying to programmatically move an added component to a specific position in the parent since I have child components, that are supposed to...
rhoenerSBS
I initially wanted to realize the same behavior with a function in the droppable property of the parent component to prevent components being dropped at a specific index but unfortunately the index is not given as a param for that function.
GJS Helper
Thank you for reporting this issue. You've identified a subtle timing bug in GrapesJS related to how the Layers panel updates when a component is programmatically moved immediately after being added. Root Cause The issue stems from a timin...
Reproducible demo link https://node-projects.github.io/web-component-designer-demo/index.htmlDescribe the bug I started a Framework to design webpages with webcomponents a few years ago, and now I found GrapeJS. Thought maybe we have the same goals and could benefit from each other? (Don't know how atm., but maybe) (l...
GJS Helper
Thank you for reaching out and for your interest in GrapesJS! It's great to see other innovative projects in the web design space. Your Web Component Designer looks very interesting, especially with its strong focus on web components, adva...
GrapesJS version[X] I confirm to use the latest version of GrapesJSWhat browser are you using? Arc Version 1.35.0 / Chromium Engine 123.0Reproducible demo link https://grapesjs.com/demo.htmlDescribe the bug My special case: In my case, all styles from plugins and general styles are located in the header. So, to achiev...
artf
it is intended to store canvas styles, styles that do not affect the result of hmtl/css, styles that will not be exported, right? Correctby default the body has a white background, this seems to be the default color in all browsers, no? ye...
artf
There is canvasCss option for that. Next time open a Discussion please, as this is not a core bug but wrong usage.
ctrhub
Hey @artf. Thanks for the answer. I'm not sure that we understood each other and I would like to clarify one detail. Yes, I know about the canvasCss property, moreover, I use it to change the styles of the selected component. I understand...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome v122 Reproducible demo link https://grapesjs.com/docs/getting-started.html#style-manager Describe the bug How to reproduce the bug?Go to the "StyleManager" section of the "Getting Started" doc from GrapeJSTry to use...
bernesto
Hi @artf, The root issue is that we are trying to place a popover palette absolutely positioned over another element at an arbitrary location on the page. And we need to do this even when the origin element is outside of the editor contain...
artf
@bernesto can you check this one as it seems to be related to your changes
bernesto
Yes, I see the issue. The logic needs to account for when the parent is outside of the container. Let me see what I can do there.
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Edge v122 Reproducible demo link https://jsfiddle.net/bwreyq29/1/ Describe the bug How to reproduce the bug? open this link https://jsfiddle.net/bwreyq29/1/ and javascript code attached to src attribute will be executed au...
bernesto
I think the pre-parser option is a really good idea. It sticks to the 'plug-in' per feature concept. How about updating fromElement to accept a string element ID or boolean. If bool == true, works as it does now, parsing the container HTML...
artf
Totally agree with @bernesto indeed no matter how hard we try to make it safe, it will never be enough and I don't want to give the impression that the library is "so safe" to justify a missing server-side validation. The current options (...
bernesto
This is unavoidable when using fromElement to load from an active DOM element. The element of the page loads and executes synchronously. GrapesJS would never have a chance to process and disarm the XSS html. This would need to be addressed...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome v122 Reproducible demo link https://jsfiddle.net/Lk2bwhp4/ Describe the bug How to reproduce the bug?Place the editor inside of a container, then programmatically resize the container's width. What is the expected b...
artf
Thanks @bernesto for the report but as we can't detect programmatic layout changes without impacting the performances, in this case, you have to update the editor layout manually via editor.refresh() (eg. in your example you would put the...
bernesto
Hi @artf, I actually had this accidentally mixed in the last pull request you accepted (src/canvas/model/Frame.ts). You might want to take a look at what I added and see if it is causing an issue. We have it in testing right now and haven'...
artf
I actually had this accidentally mixed in the last pull request you accepted and I actually removed it cause it seemed to be unrelated π€£ Not a problem to add it back but consider that your example will work only because of the desktop Dev...
GrapesJS version[X] I confirm to use the latest version of GrapesJSWhat browser are you using? chromeReproducible demo link Cannot provide it data is sensitiveDescribe the bug How to reproduce the bug?Drop a page in the editor/Canvas.Set height for ex- 600pt and also give padding of 10 pt to all 4 sides.Now drop two s...
upadhyay8844
@artf can you please have a look at it ?
GJS Helper
The issue you're encountering, where a child section with height: 100% overlaps its parent when the parent has fixed height and padding, and other siblings are present, is a common CSS layout challenge. It's not a bug in GrapesJS itself, b...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Firefox 122.0.1 Reproducible demo link https://grapesjs.com/demo-mjml.html Describe the bug How to reproduce the bug? In the official demo: https://grapesjs.com/demo-mjml.htmlAdd a column to the canvasSelect the section Go...
artf
Probably related to the mjml plugin, not the core as I can't reproduce it (I'd suggest opening an issue to the related repo).
GJSBlock
Thanks for reporting this, @codei20. Great question about Undo not working while seleting background or text color through color picker. The recommended approach with Canvas is to use the event-driven API. Start here: Check the GrapesJS do...