Hacker News Reader

Hacker News

Comment by xg15 | Hacker News Reader

Parent Full thread

xg15

window.$ = document.querySelectorAll

View on HN

Uncaught TypeError: 'querySelectorAll' called on an object that does not implement interface Document.

Damn, you're right, sorry.

window.$ = (x => document.querySelectorAll(x))

I find this a little cleaner:

    window.$ = document.querySelectorAll.bind(document);

Since it works properly for any function no matter the number of arguments it receives

I like wrapping it in an Array.from() so you can use .map/.filter/etc.

The wrong syntax notwithstanding, this doesn't let you recursively use querySelector(All), e.g. to find children of a node like document.querySelector("#foo").querySelectorAll(".bar")

I know, it was a bit of a joke.

But I think the OP's jQuery replacement is also dropping features in the service of a small footprint. So this was my 80/20 contribution to the "smallest jQuery replacement" problem ;)

But it doesn't do chaining and you have to loop through elements to do anything with them.

I'm always surprised that an API that is defined by matching 0-n dom elements doesn't return a container that by default maps over them list monad style.

There’s a fairly small polyfill that makes a DOMNodeList have the same functions as Array.

Are the various browser JS implementations clever enough not to make a new Object for Array.from(DOMNodeList) ?

window.$ = document.querySelectorAll.bind(document)

Reply on news.ycombinator.com