Skip to content

fix text nodes in .innerHTML-optimized output#3030

Merged
Conduitry merged 1 commit intomasterfrom
gh-2745
Jun 18, 2019
Merged

fix text nodes in .innerHTML-optimized output#3030
Conduitry merged 1 commit intomasterfrom
gh-2745

Conversation

@Conduitry
Copy link
Copy Markdown
Member

Fixes #2745, and also fixes another bug I discovered while looking at that one. The .innerHTML representation of 'raw' (non-top-level script and style) tags wasn't escaping the character it needed to for use in string templates.

- collapse whitespace to single space when appropriate (#2745)
- escape template string characters in script and style tags
@Conduitry Conduitry merged commit 9bce3fc into master Jun 18, 2019
@Conduitry Conduitry deleted the gh-2745 branch June 18, 2019 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Spaces between html tags (like \n\t\t\t)

1 participant