throttle function implements as debounce

[zepster]

https://github.com/impress/impress.js/blob/master/src/lib/util.js#L65-L74

        var throttle = function( fn, delay ) {
            var timer = null;
            return function() {
                var context = this, args = arguments;
                window.clearTimeout( timer );
                timer = window.setTimeout( function() {
                    fn.apply( context, args );
                }, delay );
            };
        };

[henrikingo]

Sorry, what is the issue you're raising here?

[zepster]

var throttle works like debounce
If is ok for you - you should rename throttle to debounce.
If you expect throttle behavior - function is not implemented it

https://stackoverflow.com/questions/25991367/difference-between-throttling-and-debouncing-a-function

[henrikingo]

I see. Thanks for the link.

It's actually a good question which behavior is "correct" or wanted here. It's subjective I guess, and also back in 2012 there may have been a higher priority to just resize once (debounce) whereas looking at this today I wouldn't mind throttle behavior either, as it would allow to drag the border of a browser window many seconds and see the presentation follow behind.

Interestingly, the code comment references the very same blog post from Remy Sharp as your StackOverflow answer: https://remysharp.com/2010/07/21/throttling-function-calls Clearly the function body is the debounce function from that blog post, yet for some reason the creator of impress.js chose to call it throttle.

I think it's ok to rename as debounce, to be semantically correct, and thanks for opening this issue.