GrapesJS version[X] I confirm to use the latest version of GrapesJSWhat browser are you using? Chrome, IOSReproducible demo link https://www.lovepoptestsite.com/products/disney-s-hocus-pocus-black-flame-candle-pop-up-card?variant=40356218077273&preview_link=124907487321 password: ilovecardsDescribe the bug How to repr...
loclovepop
screencast-www.lovepoptestsite.com-2023.06.01-001711.webm This is video of bug
artf
I see the touch plugin works properly, ensure to have the mobile device enabled on load (eg. not simulated post load) so the polyfill could be loaded properly.
GJS Helper
The issue you're experiencing with drag-and-drop not working on mobile screens when GrapesJS is embedded in your "test store" environment, despite using grapesjs-touch, is a common problem in complex web applications. It typically stems fr...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? ---- Reproducible demo link https://codesandbox.io/p/sandbox/pensive-browser-mfhxp8 Describe the bug How to reproduce the bug?Create Angular ProjectAdd grapeJS dependencyinclude "allowSyntheticDefaultImports": true on tsco...
marktamis
I copied over some of the settings of the tsconfig from a working vite gjs project into my stenciljs project and now it is running correctly. I think you need to do some tweaking to your angular tsconfig and it should not throw the error a...
Deepakanandrao
"skipLibCheck": true, did the trick. Thank you @marktamis ๐
artf
Yeah "skipLibCheck": true is necessary here until I find the fix for that TS output ๐
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? all and none, the bug is during the build Reproducible demo link unnecessary Describe the bug How to reproduce the bug? Use NPM to install GrapesJS to use it in a typescript project. According to the documentation, to set...
artf
This seems to be related to https://github.com/microsoft/TypeScript/issues/35945 As a workaround you could do something like this for now: I guess for the TS type safety we would need to introduce some kind of helper for plugins
jchautreux
that sounds to be a good idea
artf
Added the helper here (I'll also update the docs with the usage later)
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? all and none, the bug is during the build Reproducible demo link unnecessary Describe the bug How to reproduce the bug?create a blank projectuse NPM to install GrapesJS and a plugin (preset-webpage in this example but "wor...
artf
I'm closing this as related to https://github.com/GrapesJS/grapesjs/issues/5149
jchautreux
That's not really the same problem, in the #5149, the bug is related on how to set plugin options during the tool initialization. Here, the bug is related to the last update of the project that has defined a new index.d.ts that is not comp...
artf
In the previous release index.d.ts had the same signature for plugins so I'm not sure to what change you're referring to https://github.com/GrapesJS/grapesjs/blob/095466c8cc8034e54888d67beef4cb9296c823e4/index.d.ts#L50-L53 Can you share th...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? All Reproducible demo link None Describe the bug The definition of the function located here: https://github.com/GrapesJS/grapesjs/blob/v0.21.1/src/panels/index.ts#L82 requires the data given as the full panel instance its...
duclet
@artf - You didn't read the full sentence. You can clearly see in the code itself, it requires the argument given as the full Panel object. In your docs (as linked), you are not giving it the full Panel object in the example, but rather ju...
artf
The documentation on the site says the parameters given can be only an object with the properties Sorry, where exactly do you read that statement?
GJS Helper
Thank you for reporting this and bringing it to our attention. It's important to ensure clarity between type definitions and documentation. After reviewing the GrapesJS v0.21.1 source code and the documentation, it appears there isn't a di...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome Version 113.0.5672.63 Reproducible demo link https://jsfiddle.net/rajevar/zLs8q97f/4/ Describe the bug This is in reference to this discussion https://github.com/GrapesJS/grapesjs/discussions/4759 How to reproduce t...
rajevar
sorry, my bad. I wasn't using the latest.
artf
@rajevar I can't reproduce it with the latest version, are you sure you're using the latest one?
GJS Helper
You've identified a common synchronization issue in GrapesJS related to how component attributes are managed, especially when classes are modified via the Style Manager and then other traits (like id or title) are updated for the first tim...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? All Reproducible demo link None Describe the bug How to reproduce the bug?Per the definition here: https://github.com/GrapesJS/grapesjs/blob/dev/src/stylemanager/index.ts#L304, you should be providing the options as option...
SachinPuthiyoth
Seems like the type definition in StyleManagerConfig.sectors is unchanged : https://github.com/GrapesJS/grapesjs/blob/5c890449a52f526bc480d6dc649ff5e01ec59140/src/stylemanager/config/config.ts#L8 Shouldn't we be updating this to use Proper...
GJS Helper
Thank you for reporting this issue. Upon reviewing the GrapesJS source code, specifically the dev branch and the latest stable release (0.21.10), it appears that the PropertyProps interface does include the options property. Therefore, the...
GrapesJS version [x] I confirm to use the latest version of GrapesJS What browser are you using? Chrome v91 Reproducible demo link https://grapesjs.com/demo.html Describe the bug I want to trigger rte with a command ,i found enableEditing but it doesnt work ,is there is any command to trigger rte Code of Conduct [X] I...
GJS Helper
The issue you're encountering is that enableEditing is not a public command or method intended for programmatically triggering the Rich Text Editor (RTE) in GrapesJS. The RTE in GrapesJS is typically enabled automatically when a text compo...
GrapesJS version[X] I confirm to use the latest version of GrapesJSWhat browser are you using? ChromeReproducible demo link https://jsfiddle.net/sdimitrenco/rn8u4pc0/7/Describe the bug I'm not sure it's a bug or not, but for me, behavior look strange I have a component List that includes, component ol and component li...
artf
If the property is of the same value as the default definition, it's skipped. This is controlled by avoidDefaults config option and it's highly recommended to keep it true to have small JSONs as possible.
GJS Helper
This behavior is not a bug but an intentional optimization within GrapesJS, which leverages Backbone.js models. When you serialize GrapesJS components to JSON, attributes that match their default values for that specific component type are...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Version 1.51.110 Chromium: 113.0.5672.77 (Offizieller Build) (64-Bit) Reproducible demo link https://grapesjs.com/docs/getting-started.html#add-blocks Describe the bug On Linux Wayland (Arch Linux and Gnome) using Brave yo...
artf
By checking with the same Brave version (on Mac), drag and drop seems to work properly, so it doesn't look like an issue with GrapesJS itself or the browser. If anybody else experience the same or have more context about the issue, please...
gxanshu
I'm facing the same issue with tauri. they use webgtk and drag & drop are not working there as well
putzwasser
By checking with the same Brave version (on Mac), drag and drop seems to work properly, so it doesn't look like an issue with GrapesJS itself or the browser. I don't feel this is a valid check. You got a completely different setup by using...