fix: support repeated :scope

dperini / nwsapi

Fast CSS Selectors API Engine

MIT License

105 stars 36 forks source link

fix: support repeated :scope #35

Closed pkra closed 4 years ago

pkra commented 4 years ago

Avoids throw on selector with repeated :scope.

nwsapi.js: adjusts makeref() to replace all matches of :scope instead of just the first match.
scope03b.html: add test case

Fixes #33

pkra commented 4 years ago

As per https://github.com/dperini/nwsapi/issues/33#issuecomment-603451027, I checked nwsapi/test/w3c/dom/nodes/Element-closest.html. The resuls are identical after this change: all tests in that pag pass except for the last test (complex :has) which fails in both browser and nwsapi.

dperini commented 4 years ago

Merged pull request from @pkra which fixes issue #33