GrapesJS Issues

JGiard

I've created a pull-request with what I would expect a fix to look like, given my understanding of the issue.

artf

No, you should never rely on something like this, or you destroy and reinit, or you simply hide and show the editor.

GJSBlock

Thanks for reporting this, @JGiard. Great question about Duplicated video block when re-attaching GrapesJs editor to document. The recommended approach with Canvas is to use the event-driven API. Start here: Check the GrapesJS documentatio...

marcosvnmelo

I keep having this problem in version 0.17.19

marcosvnmelo

I'm sorry I just spelled it wrong. I meant 0.17.29.

artf

Thanks for the report @inventorbit I can see an issue in parsing the <path> tag. I'll investigate and try to fix it for the next release.

GJSBlock

Thanks for reporting this, @heminei. Great question about JS error when press "s" or "w" key - Cannot read properties of undefined (reading 'indexOf'). The recommended approach with Canvas is to use the event-driven API. Start here: Check...

arthuralmeidap

hello @flowize-project ! hope you are good! In your issue you have posted GrapesJS demo link as reproducible link but the video was recorded with a custom grapesjs installation. Could you try to record a video and reproduce the issue with...

GJSBlock

Thanks for reporting this, @flowize-project. Great question about Firefox issue when the user clicks on different component very fast. The recommended approach with Canvas is to use the event-driven API. Start here: Check the GrapesJS docu...

artf

the getHtml has nothing to do with the DOM of the canvas... Read carefully the documentation https://grapesjs.com/docs/modules/Components.html and try to understand the difference between the model and the view of the component.

mihir-khandekar

@artf thanks for the prompt response, what is the right way of doing this then? My requirement is: I want to add a component at the current cursor position. How can I then get it to be part of the editor.getHtml() method?

ionutmiftode

@mihir-khandekar did you manage to fix this?

throne1986

@artf can you please help me with this?

artf

The issue is here You can also remove that global listener and update the video Block with activate: true, way more cleaner.

throne1986

> The issue is here> > > > > > > > > > > > > You can also remove that global listener and update the video Block with activate: true, way more cleaner.Hi @artf changing as u suggested but still when I drag the video block and select a vide...

mattnoth

I didn't noticed that you had closed my previous issue -- with the weird use case. Do you have any suggestions for a work around here? I've been using the updateRte event listener, putting the RTE inside the canvas dynamically is proving a...

artf

@mattnoth I already replied to your discussion here, please avoid opening the same issue again.

GJSBlock

Thanks for reporting this, @mattnoth. Great question about RTE Overlapping text In Basic Text Component when Width & Height are 100% of Canvas. The recommended approach with ProseMirror is to use the event-driven API. Start here: Check the...

artf

Not a bug, just a weird use case.

mingxin-yang

@artf I want to change the position of rteToolbar, but pos does not have the parameters mentioned in the document

GJSBlock

Thanks for reporting this, @mattnoth. Great question about RTE overlapping text when text component set to 100% height. The recommended approach with ProseMirror is to use the event-driven API. Start here: Check the GrapesJS documentation...

throne1986

I solved the issue for future reference first I removed this function : editor.on("component:add", (model, argument) => { if (model.getName() === "Image") { modal.setContent(ModalImages); } }); I replaced with this function editor.Commands...

artf

when you run editor.command.run('open-assets') you have to pass options, eg.

throne1986

@artf I solved the issue it was about closure problem , the solution just passing the URL as options as follows editor.Commands.run("open-assets", { target: editor.getSelected() url: imageUrl }); editor.Commands.add("open-assets", { run(ed...

amansharmaagami

@artf Thanks for making GrapesJS.

GJSBlock

Thanks for reporting this, @amansharmaagami. Great question about Want to Add fixed sized height of canvas. The recommended approach with Canvas is to use the event-driven API. Start here: Check the GrapesJS documentation for your specific...