Open ivancuric opened 7 years ago
Agreed! It's a difficult thing to change now that its there, but what about adding an option and making the default an actual debounce? @pl12133 called this out to me a couple weeks ago, I'd forgotten about the name mixup.
The sooner you do it the better if you see a future for this package. We have enough misnamed "oh-it's-too-late-now" stuff in the web dev industry as it is :)
Anyway, decided to use https://www.npmjs.com/package/core-decorators instead for the project I'm working on since I do need both. The implementations are quite similar as well.
Oh nice, I hadn't seen that module! Looks good.
I'd love to have a real debounce option here somehow. Separating throttle and debounce would make a lot of sense. Also core-decorators seems to have deprecated their debounce and throttle decorators for some reason
In the docs its says:
What you're doing is throttling, not debouncing. lodash.throttle:
lodash.debounce:
And while on the topic, a (real)
@debounce
would be a great addition.