reloadDebounce ignored when using browserSync.reload with gulp

[fracz]

Having the following gulp configuration

var browserSync = require('browser-sync').create();
gulp.task('bs', ['watch'],  function() {
  browserSync.init({
    open: no,
    reloadOnRestart: yes,
    reloadDebounce: 10000,
    online: no
  });
  gulp.watch('dist/modules/**/*.{js,html}').on('change', browserSync.reload)
});

the browser is immediately refreshed after change. It should be refreshed after 10s of "event silence", as docs say.

• Browsersync [ 1.14.3 ]
• Node [ 5.1.1 ]
• Npm [ 3.3.12 ]

Affected platforms

• linux
• windows
• OS X
• freebsd
• solaris
• other (please specify which)

Browsersync use-case

• API
• Gulp
• Grunt
• CLI

[shakyShane]

@fracz thanks, I was waiting for someone to pick up on this before acting on this.

The core problem is that all logic for delay/throttle/debounce is applied to file-watching only at present. But as you've noticed, because the API method is named reload, you would expect reload* options to affect this, which they don't.

[fracz]

Apparently, the reloadDelay is respected when calling the browserSync.reload(). Although they are not the same, it can be used as a workaround for now.

[shakyShane]

ok that's good, deffo needs fixing though. there needs to be some re-structuring put in place, which is unfortunately not a quick job :(