Cavemanon/piratepoet
8
Fork 0
Code Pull Requests Actions Releases 16 Activity

Fix debouncing for ALC refresh to not update multiple times immediately

Browse Source
This commit is contained in:
Tautvidas Sipavičius
2016-08-24 16:15:53 +03:00
parent f004bb5368
commit bdab0c12fa
1 changed files with 8 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
assets/js/src/newsletter_editor/blocks/automatedLatestContent.js
View File

@@ -35,7 +35,12 @@ define([
Module.ALCSupervisor = SuperModel.extend({ Module.ALCSupervisor = SuperModel.extend({
initialize: function() { initialize: function() {
this.listenTo(App.getChannel(), 'automatedLatestContentRefresh', this.refresh); var DELAY_REFRESH_FOR_MS = 500;
this.listenTo(
App.getChannel(),
'automatedLatestContentRefresh',
_.debounce(this.refresh, DELAY_REFRESH_FOR_MS)
);
}, },
refresh: function() { refresh: function() {
var models = App.findModels(function(model) { var models = App.findModels(function(model) {
@@ -107,9 +112,7 @@ define([
this.on('change:amount change:contentType change:terms change:inclusionType change:displayType change:titleFormat change:featuredImagePosition change:titleAlignment change:titleIsLink change:imageFullWidth change:showAuthor change:authorPrecededBy change:showCategories change:categoriesPrecededBy change:readMoreType change:readMoreText change:sortBy change:showDivider', this._scheduleFetchPosts, this); this.on('change:amount change:contentType change:terms change:inclusionType change:displayType change:titleFormat change:featuredImagePosition change:titleAlignment change:titleIsLink change:imageFullWidth change:showAuthor change:authorPrecededBy change:showCategories change:categoriesPrecededBy change:readMoreType change:readMoreText change:sortBy change:showDivider', this._scheduleFetchPosts, this);
this.listenTo(this.get('readMoreButton'), 'change', this._scheduleFetchPosts); this.listenTo(this.get('readMoreButton'), 'change', this._scheduleFetchPosts);
this.listenTo(this.get('divider'), 'change', this._scheduleFetchPosts); this.listenTo(this.get('divider'), 'change', this._scheduleFetchPosts);
this.on('add remove update reset', function(model, collection, options) { this.on('add remove update reset', this._scheduleFetchPosts);
App.getChannel().trigger('automatedLatestContentRefresh');
});
this.on('refreshPosts', this.updatePosts, this); this.on('refreshPosts', this.updatePosts, this);
}, },
updatePosts: function(posts) { updatePosts: function(posts) {
@@ -120,16 +123,7 @@ define([
* ALC posts on each model change * ALC posts on each model change
*/ */
_scheduleFetchPosts: function() { _scheduleFetchPosts: function() {
var TIMEOUT = 500, App.getChannel().trigger('automatedLatestContentRefresh');
that = this;
if (this._fetchPostsTimer !== undefined) {
clearTimeout(this._fetchPostsTimer);
}
this._fetchPostsTimer = setTimeout(function() {
//that.fetchPosts();
App.getChannel().trigger('automatedLatestContentRefresh');
that._fetchPostsTimer = undefined;
}, TIMEOUT);
}, },
}); });