BUG: Too much recursion
Question
GrapesJS version
- I confirm to use the latest version of GrapesJS
What browser are you using?
Firefox 131.0.2
Reproducible demo link
https://grapesjs.com/demo-newsletter-editor.html
Describe the bug
How to reproduce the bug?
- Go to the grapesjs demo, then import the following html:
<div class="foobar">
<img data-gjs-locked="true" width="500" src="https://cdn.pixabay.com/photo/2021/12/16/15/26/forest-6874717_1280.jpg">
</div>
- Open your browser console.
- Rapidly click on the image multiple times.
What is the expected behavior? Should work without errors.
What is the current behavior? This causes a recursion error in FireFox. Chrome does not seem to be affected by this issue. I am testing this on OSX 13.6.9 and I have included a short screen recording.
Uncaught InternalError: too much recursion
getList Component.ts:1931
ensureInList Component.ts:1866
ensureInList Component.ts:1880
Ke Underscore
Backbone Backbone
ensureInList Component.ts:1880
ensureInList Component.ts:1880
Ke Underscore
Backbone Backbone
ensureInList Component.ts:1880
ensureInList Component.ts:1880
My suspicion is that the issue is coming from the following function: https://github.com/GrapesJS/grapesjs/blob/dev/packages/core/src/dom_components/model/Components.ts#L373
Thanks.
https://github.com/user-attachments/assets/54b6c40f-d435-41f9-a09f-47450f01523e
Code of Conduct
- I agree to follow this project's Code of Conduct
Answers (3)
Hey @jlafosse thanks for the report.
I'd not expect that issue with the latest version: <img width="123" alt="Screenshot 2024-10-21 at 19 00 04" src="https://github.com/user-attachments/assets/ff074a31-cd19-4547-bb27-596c92e16d64">
Can you try to check again, please?
Indeed, that appears to have fixed the issue! Thank you!
🙌 closing then, thanks 🙇♂️
Related Questions and Answers
Continue research with similar issue discussions.
Issue #6271
BUG: Drag and drop of components doesn't work on latest Firefox
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Firefox 131.0 (64-bit) on Linux and Mac Re...
Issue #4503
BUG: Missing inline styles used by CKEditor when adding raw html components
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chromium v96 Reproducible demo link https:...
Issue #4859
BUG: Editing CSS property sets a style to a common class instead of element
GrapesJS version[X] I confirm to use the latest version of GrapesJSWhat browser are you using? Chrome v108Reproducible demo link https://gr...
Issue #4732
BUG: Tabs behaviour after HTML import
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? Chrome 107 (MacOS) Reproducible demo link...
Paid Plugins That Match This Issue
Curated by issue keywords and label relevance to help you ship faster.
Loading paid plugin recommendations...
Browse Plugin Categories
Jump directly to plugin category pages on the marketplace.