GrapesJS Issues

artf

Sorry but you have to open the bug issue properly by following all steps and providing a reproducible demo.

skru

see #4473

artf

Fixed here https://github.com/artf/grapesjs/commit/f61222ec904ed56794f7b97e362ea2006dfceaf6

artf

Thanks @skru I'll take care of it

skru

first, you're code needs to go in a plugin https://grapesjs.com/docs/modules/Components.html#define-custom-component-type and you're not updating the model of the component, just the html in the canvas... https://jsfiddle.net/jmweg8u1/4/

artf

Yeah correct, thanks @skru You can also connect the content directly to the trait to read also the initial value.

artf

@mickeyDominic you're placing all the default panels under the same object Where it should be one object per panel

mickeyDominic

Thank you @artf. Silly me :)

skru

I've tried to find a fix for this but failing. Problem occurs with: onKeydown(event) { if (event.key === 'Enter') { this.doc.execCommand('insertLineBreak'); event.preventDefault(); } } Which, when creating a list is not the desired behavio...

artf

I'm not sure yet about the best long-term solution but as a workaround, we can prevent that condition but checking the active state of insertOrderedList/insertUnorderedList. I'll take care of it for the next release.

jack2ky

Where to find other commands like "insertUnorderedList"?

artf

Hi Ronaldo, I'm not sure if there is a simple way to fix this behavior globally for all use cases as you're resizing the image for both sizes and not only the width (in that case the height wouldn't change). What I can suggest trying is to...

gustavohleal

Hi @artf. One way we thought to fix this for our use case based on our users is to change the units for width and height always together. That means when the user changes the width from pixels to %, for example, the height will also be cha...

ronaldohoch

Let me know if that works for you Hi Arthur, these configuration didn't make any changes to the image, it keeps setting the height with px instead of auto, as property keepAutoHeight suggest

artf

Hi Barry, yeah currently this is the expected behavior (check also this discussion) but I hope one day to add proper support for inherited properties.

barryross

okay, thank you for letting me know On Mon, Jul 18, 2022 at 9:29 AM Artur Arseniev @.> wrote: Hi Barry, yeah currently this is the expected behavior (check also this discussion <https://github.com/artf/grapesjs/discussions/4245>) but I hop...

artf

Thanks @skru yes, there is an internal issue I have to fix to properly handle extended text components but once it's fixed another thing to keep in mind when you extend text components, it's to use components property instead of content.

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