[Discussion] ignoreWhitespace set to true rather than false?

[siddMahen]

I don't particularly see any benefit to not having this option set. Is there any reason this wasn't set already?

[matthewmueller]

Hmmm... I was left it in so that rendering was as close to the original as possible. I think if we were to ignore it, we'd need to provide an option to run the output through an HTML tidy.

[siddMahen]

I think we should ignore white spaces as they create unnecessary overhead when filtering text nodes, traversing etc, and implement a tidy output option like you suggested.

[siddMahen]

Also, I'm not sure if this is consistent with jQuery/intended behavior but with this option on, text nodes are often returned by next() and prev() methods, rather than the next element.

[siddMahen]

See #27