grapesjs strips   entities (mostly inserted by CKEditor)

[arachnosoft]

Hi @artf ,

We have just noticed that, if you import this code into grapesjs:

`My label field 1 

My label field 2` With an ` ` between the two label tags, the ` ` is being stripped from the code, leaving the two tags "glued" together. I reproduced the issue with the Newsletter demo:  If I put anything else between the two (even, sometimes... real spaces), everything is kept. Do you know if there's any explicit ` ` removal code somewhere? I looked into grapes.js and found only a reference to issue #1362 ,but it doesn't seem to have anything to do with my current problem: `// Change some spaces to NBSP to prevent the browser from collapsing` `// trailing spaces at the end of a line when rendering text (issue #1362).` `function splitSpaces()` Issue #180 sounds rather close; You told the OP to add a space between all his tags to fix his issue, but the OP replied that it was actually fixed by an update... And I tested this as well on your demo (on which, I suppose, CKEditor should be rather up-to-date, not to mention grapesjs itself) and the "space between tags" trick doesn't prevent grapesjs from stripping ` ` in my case... Maybe I could try to prevent this by catching ` `, through isComponent(), but I'd like to read your opinion on this first, in case it'd be a real bug somewhere, if not an explicit choice from you. Thanks! -Maxime

[artf]

I think the issue might be here https://github.com/artf/grapesjs/blob/0a122cd7b558bcf391cfd7e4608dffad3ba9de53/src/parser/model/ParserHtml.js#L169-L172 unfortunately nodeValue returns already processed string, without   Try to use this option: https://github.com/artf/grapesjs/blob/0a122cd7b558bcf391cfd7e4608dffad3ba9de53/src/editor/config/config.js#L104-L105

[arachnosoft]

Thank you very much for the tip @artf , this (undocumented?) Parser config option seems to solve my issue! Of course, I had to set it to true, not 0, as I wanted to keep those empty spaces ;)

And had to set it on the editor.Parser.getConfig() object. Setting this option through grapesjs.init() has no effect.

I'm closing the issue! But you should add this on your documentation ;)

[lock[bot]]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.