fix getElementsByClassName

[Hunam6]

Fixed getElementsByClassName with multiple classes separated by a space character as argument

import {DOMParser} from "../deno-dom/deno-dom-wasm.ts"

const doc = new DOMParser().parseFromString('<p class="a b c"></p>', 'text/html')!
console.log(doc.getElementsByClassName('a b')[0]!)

Before only this worked

console.log(doc.getElementsByClassName('a')[0]!)

But as described here, the first example should work too.

[b-fuze]

Multiple class names are tested with AND relation, whereas your implementation seems to test them with OR relation 🤔

[b-fuze]

I'd also prefer if you could use for .. of over .forEach

[Hunam6]

I fixed the issue you mentioned!

In addition I have two suggestions/comments/questions:

• What about adding a utils.ts file somewhere? This would avoid having such a duplicated code.
• Also what about formatting? I've made some tweak to only modify the code of the function, but every time I save the file, Deno formatter (from official Deno VSCode extension) wants to format your code because it isn't formatted as Deno's standards suggests/enforces (ref).

[b-fuze]

Formatting can be handled in a later PR maybe... Not in this one. I do need to deno fmt Deno DOM soon...

[Hunam6]

Thanks for the improvement idea! I had tunnel vision on my implementation :)

[b-fuze]

Alrighty, thanks