forked from ashi009/node-fast-html-parser
-
Notifications
You must be signed in to change notification settings - Fork 117
Permalink
Choose a base ref
{{ refName }}
default
Choose a head ref
{{ refName }}
default
Comparing changes
Choose two branches to see what’s changed or to start a new pull request.
If you need to, you can also or
learn more about diff comparisons.
Open a pull request
Create a new pull request by comparing changes across two branches. If you need to, you can also .
Learn more about diff comparisons here.
base repository: taoqf/node-html-parser
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v7.0.1
Could not load branches
Nothing to show
Loading
Could not load tags
Nothing to show
{{ refName }}
default
Loading
...
head repository: taoqf/node-html-parser
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v7.0.2
Could not load branches
Nothing to show
Loading
Could not load tags
Nothing to show
{{ refName }}
default
Loading
- 12 commits
- 9 files changed
- 4 contributors
Commits on Jan 10, 2025
-
Configuration menu - View commit details
-
Copy full SHA for 074bfdc - Browse repository at this point
Copy the full SHA 074bfdcView commit details
Commits on Apr 20, 2025
-
Configuration menu - View commit details
-
Copy full SHA for 336f0b2 - Browse repository at this point
Copy the full SHA 336f0b2View commit details
Commits on Jan 7, 2026
-
Configuration menu - View commit details
-
Copy full SHA for 6d73ea3 - Browse repository at this point
Copy the full SHA 6d73ea3View commit details -
fix: #294 Closing tag is missing but valid HTML is still not parseable
With this change, the valid HTML > `<body><main class=h-entry><p>hello</main></body>` is parsed correctly. Furthermore, p element close tags may be omitted according to the [spec](https://html.spec.whatwg.org/multipage/grouping-content.html#the-p-element) if: > there is no more content in the parent element and the parent element is an HTML element that is not an a, audio, del, ins, map, noscript, or video element, or an autonomous custom element. With this change, all these cases are handled correctly With the exception that there is no exclusion of autonomous custom elements so that p elements will be closed incorrectly when they are contained in such. However, it is not immediately obvious that node-html-parser supports autonomous custom elements, so this omission may not be significant.
Configuration menu - View commit details
-
Copy full SHA for 950865f - Browse repository at this point
Copy the full SHA 950865fView commit details -
Within valid.js, the test 'hillcrestpartyrentals.html should return Object with valid: false. not closing <p> tag on line 476' was implemented incorrectly. In fact the HTML is valid. All the p elements without close tags are contained in td elements and, therefore, the closing tag is not required. The return should be true.
Configuration menu - View commit details
-
Copy full SHA for a81fc46 - Browse repository at this point
Copy the full SHA a81fc46View commit details -
Test for #227 was broken by previous commit. This fixes it. All tests are now passing.
Configuration menu - View commit details
-
Copy full SHA for 51528c4 - Browse repository at this point
Copy the full SHA 51528c4View commit details -
Merge pull request #288 from olets/285-document-wrapper-node
docs: `parse()` always adds a wrapper node [#285]
Configuration menu - View commit details
-
Copy full SHA for 4deab9a - Browse repository at this point
Copy the full SHA 4deab9aView commit details -
Merge pull request #292 from fiddleplum/o-291
#291 Made id also update attributes.
Configuration menu - View commit details
-
Copy full SHA for b4ec281 - Browse repository at this point
Copy the full SHA b4ec281View commit details -
Configuration menu - View commit details
-
Copy full SHA for d9f5586 - Browse repository at this point
Copy the full SHA d9f5586View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5625286 - Browse repository at this point
Copy the full SHA 5625286View commit details -
Configuration menu - View commit details
-
Copy full SHA for d003185 - Browse repository at this point
Copy the full SHA d003185View commit details -
Configuration menu - View commit details
-
Copy full SHA for f7a190c - Browse repository at this point
Copy the full SHA f7a190cView commit details
Loading
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff v7.0.1...v7.0.2