Dustin Diaz comments

Results 118 comments of


                                            Dustin Diaz

parents() doesn't work on detached elements

yeah.... but this should work in Bonzo too

Use opacity instead of filter if the browser is IE9 or higher

cool. I'll add a workaround as done the same as [Morpheus](https://github.com/ded/morpheus/blob/master/morpheus.js#L35)

add window re-size and scroll events

this is going to be a little intense without throttling. perhaps adding the following would help ``` js function throttle(wait, fn, opt_scope) { var timeout return function () { var...

Conflict with require js

this has worked for years and quite possibly a bug with your implementation or the optimizer you're using. unless you can put together a test case i will mark this...

Change title on the readme?

ha. ok. I can change it.

Infinite loop

eep. that's not good. something clearly is not right with how that was supposed to work. i'll have a look

Having a function for lookup seems to only work for the first request

i gotcha. i'll have a look to see what we can do about that

Adds conditional to hanlde if redis becomes unavailable

Hi @michaelerobertsjr thanks for your contribution! This is a good and viable patch. Would you mind sticking to the existing style of the code. run `make lint` and be sure...

"expire" option issue

It's not a Redis, but, but maybe a library flaw of express-limiter. The fact is, you've left out a required field since you need to add `expire`

Sending a custom response status / message

currently no. but we can easily add this functionality.