Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.
And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.
.html() trims empty html attributes (class="" becomes class)
See original GitHub issueVersion: 1.0.0-rc.3
Input:
require('cheerio').load('<div class=""></div>').html();
Expected:
'<html><head></head><body><div class=""></div></body></html>'
Actual:
'<html><head></head><body><div class></div></body></html>'
jQuery Comparison:
$($.parseHTML('<div><div class=""></div></div>')).html();
// '<div class=""></div>'
I actually prefer the attributes being trimmed when empty. However, I am using Cheerio in a Jest Snapshot serializer, and would prefer it to have the least disruption as possible for my end users. So ideally it would have a setting to output attribute or attribute="" with the default being attribute="", to match jQuery’s .html(). This would allow me to pass along this option to my end users so they can pick based on preference.
Same outcome occurs when paired with htmlparser2.
Issue Analytics
- State:
- Created 4 years ago
- Comments:6 (2 by maintainers)
Top Results From Across the Web
Top Related Medium Post
Top Related StackOverflow Question
Troubleshoot Live Code
Top Related Reddit Thread
Top Related Hackernoon Post
Top Related Tweet
Top Related Dev.to Post
Top Related Hashnode Post
Cheerio uses two serializers under the hood. For HTML, it is usually
parse5’s serializer. I’m happy to pull in changes from the serializer, but it is out of scope for cheerio to solve this.Newer parse5 versions (I tested v5) generate
=""for every attribute even for boolean attributes.Same time, people doesn’t like if it is added for boolean attributes like
asyncordefer. parse5#184