TJ comments

Results 725 comments of

TJ

add .not

just to avoid a ton of function calls for big lists etc but not a huge deal either way

not using `this` context so much

yeah we've inherited a few bad things from jquery, parity is nice when it's a non-wtf thing, but I'd rather ditch the weird sides of jquery so people don't think...

Rename 'select' to 'filter' to match native method

filter is a shitty name IMO it's ambiguous, but we could alias

Rename 'select' to 'filter' to match native method

I suppose, select > filter for me but it doesn't hurt to alias

count(): simply return total number of items if no function supplied.

+1 for .length()

count(): simply return total number of items if no function supplied.

this needs a rebase apparently

mixing directly into an array?

`users = enumerable(users)` will work, I do this in the tests a lot, returns an Enumerable with the array as `.obj` as a wrapper. the wrapper is for the same...

mixing directly into an array?

i havent bothered to benchmark it but it's a large difference considering you're adding one prop, or adding ~40 each time and v8 can't inline the lookups etc. yeah it's...

mixing directly into an array?

unfortunately with how js is structured you more or less cannot cleanly extend things, just a language-level flaw, but almost every language does the same dumb things :(

Add current fullscreen state.

looks good to me except the name "fulled" is weird, maybe "enabled", so that in use it would look like `if (fullscreen.enabled) {`