Clean up unnecessary eslint-disable directives (per ESLint).

16 files changed, 0 insertions, 25 deletions

diff --git a/js/App.js b/js/App.js
index 80fbabe82..0adf291c2 100644
--- a/js/App.js
+++ b/js/App.js
@@ -1,6 +1,5 @@
 'use strict';
 
-/* eslint-disable new-cap */
 /* global __, Article, Headlines, Filters, fox */
 /* global xhr, PluginHost, Notify, Feeds, Cookie */
 /* global CommonDialogs, Plugins */
diff --git a/js/Article.js b/js/Article.js
index 1f4cf931e..f7672033c 100644
--- a/js/Article.js
+++ b/js/Article.js
@@ -1,6 +1,5 @@
 'use strict'
 
-/* eslint-disable no-new */
 /* global __, ngettext, App, Headlines, xhr, PluginHost, Notify, fox */
 
 const Article = {
diff --git a/js/CommonDialogs.js b/js/CommonDialogs.js
index 43c22b5f6..16c1f5f46 100644
--- a/js/CommonDialogs.js
+++ b/js/CommonDialogs.js
@@ -1,8 +1,5 @@
 'use strict'
 
-/* eslint-disable new-cap */
-/* eslint-disable no-new */
-
 /* global __, Notify, App, Feeds, xhr, Tables, fox */
 
 /* exported CommonDialogs */
diff --git a/js/CommonFilters.js b/js/CommonFilters.js
index 3e68168fe..54df70266 100644
--- a/js/CommonFilters.js
+++ b/js/CommonFilters.js
@@ -1,7 +1,5 @@
 'use strict'
 
-/* eslint-disable no-new */
-
 /* global __, App, Article, Lists, fox */
 /* global xhr, Notify, Feeds */
 
diff --git a/js/FeedTree.js b/js/FeedTree.js
index 86a396068..f0041640d 100755
--- a/js/FeedTree.js
+++ b/js/FeedTree.js
@@ -1,4 +1,3 @@
-/* eslint-disable prefer-rest-params */
 /* global __, define, App, Feeds, CommonDialogs */
 
 define(["dojo/_base/declare", "dojo/dom-construct", "dojo/_base/array", "dojo/cookie", "dijit/Tree", "dijit/Menu"], function (declare, domConstruct, array, cookie) {
@@ -208,7 +207,6 @@ define(["dojo/_base/declare", "dojo/dom-construct", "dojo/_base/array", "dojo/co
 			return [item.updated, item.error].filter((x) => x && x !== "").join(" - ");
 		},
 		getIconClass: function (item, opened) {
-			// eslint-disable-next-line no-nested-ternary
 			return (!item || this.model.mayHaveChildren(item)) ? (opened ? "dijitFolderOpened" : "dijitFolderClosed") : "feed-icon";
 		},
 		getLabelClass: function (item/* , opened */) {
diff --git a/js/Headlines.js b/js/Headlines.js
index 1a59ffda0..1725998bc 100755
--- a/js/Headlines.js
+++ b/js/Headlines.js
@@ -228,7 +228,6 @@ const Headlines = {
 		} else if (event.ctrlKey) {
 			Headlines.select('invert', id);
 		} else {
-			// eslint-disable-next-line no-lonely-if
 			if (App.isCombinedMode()) {
 
 				if (event.altKey && !in_body) {
@@ -268,7 +267,6 @@ const Headlines = {
 
 				return in_body;
 			} else {
-				// eslint-disable-next-line no-lonely-if
 				if (event.altKey) {
 					Article.openInNewWindow(id);
 					Headlines.toggleUnread(id, 0);
diff --git a/js/PrefFeedTree.js b/js/PrefFeedTree.js
index 92e748903..142060488 100644
--- a/js/PrefFeedTree.js
+++ b/js/PrefFeedTree.js
@@ -1,4 +1,3 @@
-/* eslint-disable prefer-rest-params */
 /* global __, lib, define, CommonDialogs, Notify, Tables, xhr, fox, App */
 
 define(["dojo/_base/declare", "dojo/dom-construct", "lib/CheckBoxTree", "dojo/_base/array", "dojo/cookie"],
@@ -143,7 +142,6 @@ define(["dojo/_base/declare", "dojo/dom-construct", "lib/CheckBoxTree", "dojo/_b
 			return rc;
 		},
 		getIconClass: function (item, opened) {
-			// eslint-disable-next-line no-nested-ternary
 			return (!item || this.model.store.getValue(item, 'type') === 'category') ? (opened ? "dijitFolderOpened" : "dijitFolderClosed") : "feed-icon";
 		},
 		reload: function() {
diff --git a/js/PrefFilterTree.js b/js/PrefFilterTree.js
index 64984b107..1f20da581 100644
--- a/js/PrefFilterTree.js
+++ b/js/PrefFilterTree.js
@@ -1,4 +1,3 @@
-/* eslint-disable prefer-rest-params */
 /* global __, define, lib, xhr, App, Notify, Filters */
 
 define(["dojo/_base/declare", "dojo/dom-construct", "lib/CheckBoxTree"], function (declare, domConstruct) {
@@ -71,7 +70,6 @@ define(["dojo/_base/declare", "dojo/dom-construct", "lib/CheckBoxTree"], functio
 			return label;
 		},
 		getIconClass: function (item, opened) {
-			// eslint-disable-next-line no-nested-ternary
 			return (!item || this.model.mayHaveChildren(item)) ? (opened ? "dijitFolderOpened" : "dijitFolderClosed") : "invisible";
 		},
 		getRowClass: function (item, opened) {
diff --git a/js/PrefHelpers.js b/js/PrefHelpers.js
index 38fa7e243..fbe1d37f5 100644
--- a/js/PrefHelpers.js
+++ b/js/PrefHelpers.js
@@ -1,6 +1,5 @@
 'use strict';
 
-/* eslint-disable no-new */
 /* global __, Tables, Notify, xhr, App, fox */
 
 const	Helpers = {
diff --git a/js/PrefLabelTree.js b/js/PrefLabelTree.js
index 1859906ee..a4540deec 100644
--- a/js/PrefLabelTree.js
+++ b/js/PrefLabelTree.js
@@ -1,4 +1,3 @@
-/* eslint-disable prefer-rest-params */
 /* global __, define, lib, xhr, Notify, fox, App */
 
 define(["dojo/_base/declare", "dojo/dom-construct", "lib/CheckBoxTree", "dijit/form/DropDownButton"], function (declare, domConstruct) {
@@ -40,7 +39,6 @@ define(["dojo/_base/declare", "dojo/dom-construct", "lib/CheckBoxTree", "dijit/f
 			return tnode;
 		},
 		getIconClass: function (item, opened) {
-			// eslint-disable-next-line no-nested-ternary
 			return (!item || this.model.mayHaveChildren(item)) ? (opened ? "dijitFolderOpened" : "dijitFolderClosed") : "invisible";
 		},
 		getSelectedLabels: function() {
diff --git a/js/SingleUseDialog.js b/js/SingleUseDialog.js
index b38b3f969..a207c0c44 100644
--- a/js/SingleUseDialog.js
+++ b/js/SingleUseDialog.js
@@ -1,4 +1,3 @@
-/* eslint-disable prefer-rest-params */
 /* global define */
 define(["dojo/_base/declare", "dijit/Dialog"], function (declare) {
 	return declare("fox.SingleUseDialog", dijit.Dialog, {
diff --git a/js/common.js b/js/common.js
index bccd0dcbb..3e8a66e5b 100755
--- a/js/common.js
+++ b/js/common.js
@@ -1,7 +1,6 @@
 'use strict';
 
 /* global App, __csrf_token */
-/* eslint-disable no-new */
 
 /* exported __ */
 function __(msg) {
diff --git a/js/form/ComboButton.js b/js/form/ComboButton.js
index 4c97809e4..98386eead 100755
--- a/js/form/ComboButton.js
+++ b/js/form/ComboButton.js
@@ -1,4 +1,3 @@
-/* eslint-disable prefer-rest-params */
 /* global define */
 define(["dojo/_base/declare", "dijit/form/ComboButton"], function (declare) {
 	return declare("fox.form.ComboButton", dijit.form.ComboButton, {
diff --git a/js/form/DropDownButton.js b/js/form/DropDownButton.js
index 357d5de91..60d8e86ef 100755
--- a/js/form/DropDownButton.js
+++ b/js/form/DropDownButton.js
@@ -1,4 +1,3 @@
-/* eslint-disable prefer-rest-params */
 /* global define */
 define(["dojo/_base/declare", "dijit/form/DropDownButton"], function (declare) {
 	return declare("fox.form.DropDownButton", dijit.form.DropDownButton, {
diff --git a/js/form/Select.js b/js/form/Select.js
index 2259fd762..bc352e612 100755
--- a/js/form/Select.js
+++ b/js/form/Select.js
@@ -1,4 +1,3 @@
-/* eslint-disable prefer-rest-params */
 /* global define */
 // FIXME: there probably is a better, more dojo-like notation for custom data- properties
 define(["dojo/_base/declare",
diff --git a/js/form/ValidationTextArea.js b/js/form/ValidationTextArea.js
index c53260f40..ced4f8fe7 100644
--- a/js/form/ValidationTextArea.js
+++ b/js/form/ValidationTextArea.js
@@ -1,5 +1,4 @@
 // https://stackoverflow.com/questions/19317258/how-to-use-dijit-textarea-validation-dojo-1-9
-/* eslint-disable no-new */
 /* global define */
 
 define(["dojo/_base/declare", "dojo/_base/lang", "dijit/form/SimpleTextarea", "dijit/form/ValidationTextBox"],
@@ -10,7 +9,6 @@ define(["dojo/_base/declare", "dojo/_base/lang", "dijit/form/SimpleTextarea", "d
                 this.constraints = {};
                 this.baseClass += ' dijitValidationTextArea';
             },
-            // eslint-disable-next-line no-template-curly-in-string
             templateString: "<textarea ${!nameAttrSetting} data-dojo-attach-point='focusNode,containerNode,textbox' autocomplete='off'></textarea>",
             validator: function(value, constraints) {
                 //console.log(this, value, constraints);