Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

normalizeWhitespace with attributes and CDATA #11

Closed
stevenvachon opened this issue Feb 25, 2014 · 7 comments
Closed

normalizeWhitespace with attributes and CDATA #11

stevenvachon opened this issue Feb 25, 2014 · 7 comments

Comments

@stevenvachon
Copy link

They should be preserved, or provide a preserveSensitiveWhitespace option.

The issue I'm having is with CSS styles within CDATA; content and URLs need those spaces.

@fb55
Copy link
Owner

fb55 commented Feb 25, 2014

  • Attributes aren't normalized.
  • I'm okay with preserving whitespace in CDATA sections.

@stevenvachon
Copy link
Author

Just to clarify, CDATA's whitespace is currently not preserved, correct? It isn't for me, anyway.

@fb55
Copy link
Owner

fb55 commented Feb 26, 2014

No, there currently isn't any distinction between container elements. Maybe preserving whitespace would be relevant for script & style-tags as well?

@stevenvachon
Copy link
Author

Ah, yes, agreed. Other parsers could be used to minify those more accurately.

@rand0mUser
Copy link

Also, it is important not to alter any whitespace inside <pre> tags

@mitar
Copy link

mitar commented Nov 5, 2015

+1

@fb55
Copy link
Owner

fb55 commented Jan 20, 2018

Closing as this is out of the scope of this library.

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

No branches or pull requests

4 participants