GrapesJS Issues

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.

ClaudeCode

Thanks for reporting this, @rhoenerSBS. Great question about Double Entry in Layers if Component is moved directly after add. The recommended approach with Canvas is to use the event-driven API. Start here: Check the GrapesJS documentation...

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...

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.

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...

upadhyay8844

@artf can you please have a look at it ?

ClaudeCode

Thanks for reporting this, @upadhyay8844. Great question about =. The recommended approach with Canvas is to use the event-driven API. Start here: Check the GrapesJS documentation for your specific module Look for the on() event listener m...

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).

ClaudeCode

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...

artf

Avoid using infinite canvas as it was never documented and not even finished as a feature.

ClaudeCode

Thanks for reporting this, @mauriciolcs2. Great question about The infinite canvas is increasing its height infinitely when there is an element with a min-height of 100vh. The recommended approach with Canvas is to use the event-driven API...

artf

The problem here is not Undomanager.remove but what is tracked. When you're resizing the image, you're not editing the Component itself but its CssRule object so that's why it's not working as expected. I'd probably suggest to simply skip...

ClaudeCode

Thanks for reporting this, @yashvi2026. Great question about Undomanager.remove() not working. The recommended approach with Canvas is to use the event-driven API. Start here: Check the GrapesJS documentation for your specific module Look...

artf

Thanks @kla-ko there is actually one logic that prevents selecting other components during component:remove events and I will fix it for the next release but what I'd suggest to you is to switch to another event. The event component:remove...

kla-ko

Thanks @artf . This works like a charme ! /

ClaudeCode

Thanks for reporting this, @kla-ko. Great question about Cannot select component in `component:remove handler. The recommended approach with Canvas is to use the event-driven API. Start here: Check the GrapesJS documentation for your speci...

artf

Good catch, will fix it

ClaudeCode

Thanks for reporting this, @kla-ko. Great question about configuring an onStart() resize callback prevents resizing. The recommended approach with ProseMirror is to use the event-driven API. Start here: Check the GrapesJS documentation for...