GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome v114 Reproducible demo link https://jsfiddle.net/rmadeiraneto/t659usxv/39/ Describe the bug How to reproduce the bug?Go to the blocks list and drag the custom component "example" to the canvasThree elements that are...
artf
Thanks @rmadeiraneto for the report. Yeah, unfortunately that's an issue if you're mutating arrays/objects properties in that way and to avoid that you have 2 options:Avoid direct mutations (assign new references when you have to update th...
rmadeiraneto
@artf using defaults as a function works, thanks for quick response and for providing the alternative fix on this issue. About the first suggestion, I think it's not about people should mutate or not the object, because we're talking about...
GJS Helper
The issue you've described is a common JavaScript pitfall related to how mutable objects (like arrays and objects) are handled when assigned as default values. In GrapesJS, when you define a component with defaults that include arrays or o...
GrapesJS version[X] I confirm to use the latest version of GrapesJSWhat browser are you using? Version 113.0.5672.126 (Build officiel) (64 bits)Reproducible demo link https://grapesjs.com/demo-newsletter-editor.htmlDescribe the bug How to reproduce the bug?Go on demo website : https://grapesjs.com/demo-newsletter-edit...
artf
Thanks @acantepie please open the issue in the proper repo as this is not core related one
GJS Helper
The issue you're encountering, where the Rich Text Editor (RTE) toolbar in GrapesJS doesn't correctly refresh its state (e.g., bold, italic buttons) when switching between multiple text components, is a known synchronization challenge. Thi...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome Reproducible demo link https://codesandbox.io/s/adoring-margulis-873kdv Describe the bug Context Move component method doesn't work as expected. This piece of code doesn't work as expected. this piece of code should...
krishnaeverestengineering
Fix for this issue: https://github.com/GrapesJS/grapesjs/pull/5176
artf
Thanks @krishnaeverestengineering but this is actually the expected behaviour for moving the component inside the same parent. This is because the at should account also for the index of the component itself. So, if you try to move "Compon...
krishnaeverestengineering
Thanks @krishnaeverestengineering but this is actually the expected behaviour for moving the component inside the same parent. This is because the at should account also for the index of the component itself. So, if you try to move "Compon...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Version 112.0.5615.137 (Official Build) (arm64) Reproducible demo link https://jsfiddle.net/yhj6qa1u/6/ Describe the bug How to reproduce the bug?Select a component and add some style using Style Manager.Use editor.Css.set...
artf
Hi @FaisalShaikhHA this is expected behaviour but I guess the documentation of setRule is a bit misleading. The rule is actually updated (it doesn't create a new one with the same selector) but styles are simply replaced. This is what you...
FaisalShaikhHA
Thanks @artf for the clarification
GJS Helper
The behavior you're observing with editor.Css.setRule is not a bug but rather how the GrapesJS CSS Composer API is designed to handle rule updates. When you call editor.Css.setRule(selector, styleObject), if a rule with that exact selector...
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Version 112.0.5615.49 (Official Build) (x86_64) Reproducible demo link https://jsfiddle.net/shomeelconovo/k7L2rthg/1/ Describe the bug I need help with something regarding the grapes.js. I have created a project and whenev...
The difference in default display properties between your GrapesJS project and the official demo stems from the CSS loaded within the GrapesJS canvas iframe, not from GrapesJS itself enforcing display: table or display: flex universally. G...
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? 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 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...
i am getting editor.getSelected().view.enableEditing(); is not a function
artf
Please avoid opening bug issues without following the issue template. You can start a new discussion with a proper description/use case of what you're looking for.
GJS Helper
The error editor.getSelected().view.enableEditing() is not a function occurs because the enableEditing method is not a public API directly available on the ComponentView instance (which editor.getSelected().view refers to). The GrapesJS ed...