Issue #6260πŸ’¬ AnsweredOpened October 24, 2024by hr12010 reactions

An error when I use component:clone in a component

Quick answerby artf

@hr1201 onRender could be triggered many times, you should never attach listeners there if you don't clean up the properly. Your logic is also purely model related so it shouldn't even be in the view, just move your code in the model.init method

Read full answer below ↓

Question

GrapesJS version

  • I confirm to use the latest version of GrapesJS

What browser are you using?

Microsoft Edge 130.0.2849.46

Reproducible demo link

https://jsfiddle.net/uwybfd98/

Describe the bug

How to reproduce the bug?

  1. select the carousel component
  2. click the space in the upper right or lower left corner of the image and select div;
  3. clone twice

What is the expected behavior? a clone is triggered only once

What is the current behavior? it recurses, and it causes a jam

If is necessary to execute some code in order to reproduce the bug, paste it here below:

onRender({ el, model }) {
        model.getChildAt(0).forEachChild(child => {
          child.on('component:clone', function (originalModel, cloneModel) {
            console.log('clone')
            if (model.getChildAt(0).get('components').length == 6) return;
            model.trigger("change:script"); 
          })
          child.on('component:remove:before', function (model1, removeFn, options) {
            console.log('remove')
            if (model.getChildAt(0).get('components').length == 2) {
              options.abort = true;
              return;
            }
            model.trigger("change:script");
          })
        })
      },

dc23c5c8e486b07c53a673e38406a1e

Code of Conduct

  • I agree to follow this project's Code of Conduct

Answers (2)

artfβ€’ November 1, 2024

@hr1201 onRender could be triggered many times, you should never attach listeners there if you don't clean up the properly.

Your logic is also purely model related so it shouldn't even be in the view, just move your code in the model.init method

ClaudeCodeβ€’ May 17, 2026

Thanks for reporting this, @hr1201.

Great question about an error when I use component:clone in a component. The recommended approach with Components is to use the event-driven API.

Start here:

  1. Check the GrapesJS documentation for your specific module
  2. Look for the on() event listener method
  3. Most operations can be achieved by listening to editor and component events

Common patterns:

// Listen for changes
editor.on('change', () => console.log('something changed'));

// Component lifecycle
editor.on('component:mount', (c) => console.log('component ready', c));
editor.on('component:update', (c) => console.log('component updated', c));

If you're still stuck:

  • Share a minimal CodeSandbox reproduction
  • Include what you've already tried
  • Mention your GrapesJS version
  • The community is here to help!

Related Questions and Answers

Continue research with similar issue discussions.

Paid Plugins That Match This Issue

Curated by issue keywords and label relevance to help you ship faster.

View all plugins

Loading paid plugin recommendations...

Free option

Check the open-source GrapesJS plugins on GitHub or run a quick search in our free catalog.

Browse free plugins β†’
Premium option

Premium plugins ship with support, regular updates, and production-ready features β€” save days of integration work.

Browse premium plugins β†’

Related tutorials

In-depth guides on the same topic.

All tutorials β†’

Browse Plugin Categories

Jump directly to plugin category pages on the marketplace.