BUG: parser ignores `parser.optionsHtml.keepEmptyTextNodes` option
Question
GrapesJS version
- I confirm to use the latest version of GrapesJS
What browser are you using?
Chrom v140.0.7307.0 (canary)
Reproducible demo link
https://jsfiddle.net/y3gr69s4/
Describe the bug
How to reproduce the bug?
- Create editor instance with
parser.optionsHtml.keepEmptyTextNodesoption settrue.const editor = grapesjs.init({ parser: { optionsHtml: { keepEmptyTextNodes: true, }, }, }); - Load HTML code that contains newlines between elements.
editor.setComponents('<body>\n<p>foo</p>\n<p>bar</p>\n</body>'); - Get HTML code.
const result = editor.getHtml();
What is the expected behavior? The resulting HTML code still contains newlines between elements.
<body>
<p>foo</p>
<p>bar</p>
</body>
What is the current behavior? Newlines between elements are stripped from the resulting HTML code.
<body><p>foo</p><p>bar</p></body>
We get the expected result if we set parser.keepEmptyTextNodes option to true but it causes TypeScript type error. I think that keepEmptyTextNodes should be a property of optionsHtml since it is irrelevant to CSS parser.
const editor = grapesjs.init({
parser: {
// Error: 'keepEmptyTextNodes' does not exist on type 'ParserConfig'
keepEmptyTextNodes: true,
},
});
Code of Conduct
- I agree to follow this project's Code of Conduct
Answers (0)
No answers yet.
Related Questions and Answers
Continue research with similar issue discussions.
Issue #6603
BUG: Ampersand is not escaped in attribute value
GrapesJS version [x] I confirm to use the latest version of GrapesJS What browser are you using? Chrome v142.0.7392.0 (canary) Reproducible...
Issue #3779
BUG: editor.getSelected().toHTML() encodes inline javascript
GrapesJS version[X] I confirm to use the latest version of GrapesJSWhat browser are you using? latest (chrome)Reproducible demo link https:...
Issue #6409
BUG: Cannot drag to move components on mobile
GrapesJS version [x] I confirm to use the latest version of GrapesJS What browser are you using? Chrome v133 Reproducible demo link https:/...
Issue #4572
BUG: Page update event doesn't fire
GrapesJS version [X] I confirm to use the latest version of GrapesJS What browser are you using? chrome Reproducible demo link https://jsfi...
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.