GrapesJS Issues

bovealexandre

so the add function is fixed there was an error with the data I sent but I still can't understand why Tailwind css doesn't apply despite the fact that it is in the head tags

artf

Are you using config.canvas.styles option? https://github.com/artf/grapesjs/blob/b657428b3efc343513f6c92eb6b418935d2a8ac9/src/canvas/config/config.js#L14-L22 Can you create a reproducible demo of your use case?

bovealexandre

Hi @artf unfortunately I can't create a reproducible demo on CodePen or JSFiddle because there are way too many things on it and I'm using technologies like nextJS but I can give you access to the repository if you need it is a private rep...

artf

Yeah, this is something we don't support currently, but I think it might be nice to handle also those cases.

GJSBlock

Thanks for reporting this, @Singwai. The issue with CSS Parser's shape doesn't allow the CSS property to be defined more than once. appears to be a race condition or state management timing problem. This typically happens when component li...

artf

I'm not sure exactly what is happening but I'd suggest switching to styleable:change event and pass its options to setStyle, eg.

GJSBlock

Thanks for reporting this, @booellean. Great question about Undo Manager creates Duplicate CSS during custom command. The recommended approach with StyleManager is to use the event-driven API. Start here: Check the GrapesJS documentation f...

GJSBlock

Thanks for reporting this, @m-jojo-s. Great question about Unable to move editable text cursor on touch. The recommended approach with GrapesJS is to use the event-driven API. Start here: Check the GrapesJS documentation for your specific...

artf

Sorry but this is not a bug, it simply doesn't work that way. Anyway refer always to the latest documentation https://grapesjs.com/docs/modules/Components-js.html#passing-properties-to-scripts as this method of passing properties was depre...

GJSBlock

Thanks for reporting this, @bimsina. Great question about Can't parse trait values using Template literals(String Interpolation). The recommended approach with Components is to use the event-driven API. Start here: Check the GrapesJS docum...

YaseenHajajweh

@artf , sorry i found the solution

GJSBlock

Thanks for reporting this, @YaseenHajajweh. The issue with StyleManager.addProperty , type: 'select' , Cannot read properties of undefined (reading 'replace') appears to be a race condition or state management timing problem. This typicall...

artf

Thanks for the report, will be fixed in the next release.

Rawne

Also running into this XSS bug. For example adding "><img src=x onerror=alert('XSS')> to the classes of a component will cause it to pop up as well.

zgeist

Thanks a lot!

GJSBlock

Thanks for reporting this, @m-jojo-s. Great question about resizer does not work with touch - draggable attribute not set on handlers. The recommended approach with GrapesJS is to use the event-driven API. Start here: Check the GrapesJS do...

artf

Sorry but the issue tracker is mainly for bugs in the core grapesjs module and I believe in this case you should instantiate the bootstrap.Carousel manually via JS (eg. by creating a custom component with JS) as the editor has to load canv...

shelendravashishtha2

Hi Arthur, is there any way of getting screen sized specific styles in grapesjs, like different styles based on media query styles ?

GJSBlock

Thanks for reporting this, @shelendravashishtha2. Great question about Bootstrap Carousel Slider is not working. The recommended approach with StyleManager is to use the event-driven API. Start here: Check the GrapesJS documentation for yo...

artf

Thanks @sohrabi2050 will fix it for the next release.

GJSBlock

Thanks for reporting this, @ghost. The issue with button traits command not working appears to be a race condition or state management timing problem. This typically happens when component lifecycle events and DOM modifications overlap, cr...