dan/node-html-parser
1
0
Fork 0
mirror of https://github.com/danbulant/node-html-parser synced 2026-05-19 04:18:52 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge pull request #1 from raffecat/master

Browse source 
Allow custom element names with multiple dashes
This commit is contained in:
taoqf 2018-08-27 12:53:02 +08:00 committed by GitHub
commit 20f65eb5de
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 11 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/index.ts
View file

@ -550,7 +550,8 @@ export class Matcher {
}
}
const kMarkupPattern = /<!--[^]*?(?=-->)-->|<(\/?)([a-z][a-z0-9]*-?[a-z0-9]*)\s*([^>]*?)(\/?)>/ig;
// https://html.spec.whatwg.org/multipage/custom-elements.html#valid-custom-element-name
const kMarkupPattern = /<!--[^]*?(?=-->)-->|<(\/?)([a-z][-.0-9_a-z]*)\s*([^>]*?)(\/?)>/ig;
const kAttributePattern = /(^|\s)(id|class)\s*=\s*("([^"]+)"|'([^']+)'|(\S+))/ig;
const kSelfClosingElements = {
meta: true,

9
test/html.js
View file

@ -272,4 +272,13 @@ describe('HTML Parser', function () {
root.firstChild.tagName.should.eql('my-widget');
});
});
describe('Custom Element multiple dash', function () {
it('parse "<my-new-widget></my-new-widget>" tagName should be "my-new-widget"', function () {
var root = parseHTML('<my-new-widget></my-new-widget>');
root.firstChild.tagName.should.eql('my-new-widget');
});
});
});