diff --git a/template/app/Settings.js b/template/app/Settings.js index 747b20f7e2aeac648679df4ea4eb6b64ee97aa2e..c7e1aff21d37bd63911faf2c5cf035e4501e6343 100644 --- a/template/app/Settings.js +++ b/template/app/Settings.js @@ -20,7 +20,6 @@ Ext.define("Docs.Settings", { }, comments: { hideRead: false, - hideCurrentUser: false, sortByScore: false }, showPrivateClasses: false, diff --git a/template/app/controller/Comments.js b/template/app/controller/Comments.js index aaf1ee717bb71060cb9b82aeb748ee318b42e971..11a2371a6306a594975971bbec9a5373629c9c93 100644 --- a/template/app/controller/Comments.js +++ b/template/app/controller/Comments.js @@ -296,7 +296,6 @@ Ext.define('Docs.controller.Comments', { offset: offset || 0, limit: 100, hideRead: settings.hideRead ? 1 : undefined, - hideCurrentUser: settings.hideCurrentUser ? 1 : undefined, sortByScore: settings.sortByScore ? 1 : undefined, username: this.recentCommentsSettings.username }; diff --git a/template/app/view/comments/List.js b/template/app/view/comments/List.js index cb7417876564b428582ff179e440e24524f5305a..636fdf451198dea14db4e562b849234871cf3cc2 100644 --- a/template/app/view/comments/List.js +++ b/template/app/view/comments/List.js @@ -16,7 +16,6 @@ Ext.define('Docs.view.comments.List', { '

Comments

', '

', '', - '', '

' ].join(" ") } @@ -68,26 +67,23 @@ Ext.define('Docs.view.comments.List', { } }, - // Initializes all checkboxes from settings. - // Bind event handlers to fire changeSetting event when checked/unchecked. + // Initializes the hideRead checkbox from settings. initCheckboxes: function() { var settings = Docs.Settings.get("comments"); - Ext.Array.forEach(['hideRead', 'hideCurrentUser'], function(id) { - var cb = Ext.get(id); - if (cb) { - cb.dom.checked = settings[id]; - cb.on("change", function() { - this.saveSetting(id, cb.dom.checked); - /** - * @event settingChange - * Fired when one of the comments settings checkboxes is checked/unchecked. - * @param {String} name The name of the setting - * @param {Boolean} enabled True if setting is turned on, false when off. - */ - this.fireEvent("settingChange", id, cb.dom.checked); - }, this); - } - }, this); + var cb = Ext.get('hideRead'); + if (cb) { + cb.dom.checked = settings.hideRead; + cb.on("change", function() { + this.saveSetting('hideRead', cb.dom.checked); + /** + * @event settingChange + * Fired when one of the comments settings checkboxes is checked/unchecked. + * @param {String} name The name of the setting + * @param {Boolean} enabled True if setting is turned on, false when off. + */ + this.fireEvent("settingChange", 'hideRead', cb.dom.checked); + }, this); + } // Hide the hideRead checkbox if user is not moderator this.setHideReadVisibility();