Issue #4058💬 AnsweredOpened January 8, 2022by clonefunnels0 reactions

Hyphen changed to " by editor messes up modern coding.

快速解答by artf

This is not a bug, but how attribute escaping works in HTML.

Read full answer below ↓

Question

GrapesJS version

  • I confirm to use the latest version of GrapesJS

What browser are you using?

Chrome

Reproducible demo link

Not necessary - Obvious issue

Describe the bug

This is what the code needs to be:

data-zanim-lg='{"from":{"opacity":1,"x":70},"to":{"opacity":1,"x":0},"ease":"CubicBezier","duration":0.8,"delay":0.3}' data-zanim-xs='{"from":{"opacity":1,"y":-37},"to":{"opacity":1,"y":0},"ease":"CubicBezier","duration":0.8,"delay":0.3}'

This is what the editor changes it to:

data-zanim-lg="{& quot ; from& quot ;:{& quot ;opacity& quot ;:1,& quot ;x& quot ;:70},& quot ;to& quot ;:{& quot ;opacity& quot ;:1,& quot ;x& quot ;:0},& quot ;ease& quot ;:& quot ;CubicBezier& quot ;,& quot ;duration& quot ;:0.8,& quot ;delay& quot ;:0.3}" data-zanim-xs="{& quot ; from& quot ;:{& quot ;opacity& quot ;:1,& quot ;y& quot ;:-37},& quot ;to& quot ;:{& quot ;opacity& quot ;:1,& quot ;y& quot ;:0},& quot ;ease& quot ;:& quot ;CubicBezier& quot ;,& quot ;duration& quot ;:0.8,& quot ;delay& quot ;:0.3}"

This would be okay if the editor automatically changed it back when it finished but it doesn't.

Please fix the editor to allow a hyphen instead of fubarring all modern coding.

I do see that you allow hyphens after the =" so I see no reason why you can't allow them on the outside.

Code of Conduct

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

Answers (2)

artfJanuary 24, 2022

This is not a bug, but how attribute escaping works in HTML.

domEl.innerHTML = `<div data-attr='{"key": "value"}'>Text</div>`;
domEl.innerHTML;
// -> '<div data-attr="{&quot;key&quot;: &quot;value&quot;}">Text</div>'
ClaudeCodeMay 17, 2026

Thanks for reporting this, @clonefunnels.

The issue with Hyphen changed to " by editor messes up modern coding. appears to be a race condition or state management timing problem. This typically happens when component lifecycle events and DOM modifications overlap, creating an inconsistent state.

What to try:

  1. Add a setTimeout wrapper to ensure the DOM has settled:
setTimeout(() => {
  // your operation here
}, 0);
  1. Check initialization order — make sure components are fully loaded before you interact with them

  2. Use the editor's event system — listen to completion events:

editor.on('component:mount', (component) => {
  // safe to interact with component here
});

Recommended next steps:

  • Test with the latest GrapesJS version if you haven't
  • Provide a minimal reproducible example (CodeSandbox) — this helps the team identify the root cause faster
  • Include GrapesJS version, browser, and console errors in your report

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.