Update eslint spacing rules (#366)

* Add space-infix-ops rule

* Add no-multi-spaces

---------

Co-authored-by: Darius Jahandarie <djahandarie@gmail.com>

6 files changed, 16 insertions, 10 deletions

diff --git a/ext/js/pages/settings/backup-controller.js b/ext/js/pages/settings/backup-controller.js
index 94f85416..85803077 100644
--- a/ext/js/pages/settings/backup-controller.js
+++ b/ext/js/pages/settings/backup-controller.js
@@ -540,7 +540,7 @@ export class BackupController {
      * @param {string} message
      * @param {boolean} [isWarning]
      */
-    _databaseExportImportErrorMessage(message, isWarning=false) {
+    _databaseExportImportErrorMessage(message, isWarning = false) {
         /** @type {HTMLElement} */
         const errorMessageContainer = querySelectorNotNull(document, '#db-ops-error-report');
         errorMessageContainer.style.display = 'block';
diff --git a/ext/js/pages/settings/dictionary-controller.js b/ext/js/pages/settings/dictionary-controller.js
index 0d84ccbf..63671feb 100644
--- a/ext/js/pages/settings/dictionary-controller.js
+++ b/ext/js/pages/settings/dictionary-controller.js
@@ -324,7 +324,7 @@ class DictionaryExtraInfo {
                 node.parentNode.removeChild(node);
             }
         }
-        this._nodes.length =0;
+        this._nodes.length = 0;
     }
 
     // Private
diff --git a/ext/js/pages/settings/keyboard-mouse-input-field.js b/ext/js/pages/settings/keyboard-mouse-input-field.js
index f0a53f1a..99af3941 100644
--- a/ext/js/pages/settings/keyboard-mouse-input-field.js
+++ b/ext/js/pages/settings/keyboard-mouse-input-field.js
@@ -30,7 +30,7 @@ export class KeyboardMouseInputField extends EventDispatcher {
      * @param {?import('environment').OperatingSystem} os
      * @param {?(pointerType: string) => boolean} [isPointerTypeSupported]
      */
-    constructor(inputNode, mouseButton, os, isPointerTypeSupported=null) {
+    constructor(inputNode, mouseButton, os, isPointerTypeSupported = null) {
         super();
         /** @type {HTMLInputElement} */
         this._inputNode = inputNode;
@@ -65,7 +65,7 @@ export class KeyboardMouseInputField extends EventDispatcher {
      * @param {boolean} [mouseModifiersSupported]
      * @param {boolean} [keySupported]
      */
-    prepare(key, modifiers, mouseModifiersSupported=false, keySupported=false) {
+    prepare(key, modifiers, mouseModifiersSupported = false, keySupported = false) {
         this.cleanup();
 
         this._mouseModifiersSupported = mouseModifiersSupported;
diff --git a/ext/js/pages/settings/keyboard-shortcuts-controller.js b/ext/js/pages/settings/keyboard-shortcuts-controller.js
index 0734faa1..cbdae77d 100644
--- a/ext/js/pages/settings/keyboard-shortcuts-controller.js
+++ b/ext/js/pages/settings/keyboard-shortcuts-controller.js
@@ -46,6 +46,7 @@ export class KeyboardShortcutController {
         this._stringComparer = new Intl.Collator('en-US'); // Invariant locale
         /** @type {HTMLElement} */
         this._scrollContainer = querySelectorNotNull(document, '#keyboard-shortcuts-modal .modal-body');
+        /* eslint-disable no-multi-spaces */
         /** @type {Map<string, import('keyboard-shortcut-controller').ActionDetails>} */
         this._actionDetails = new Map([
             ['',                                 {scopes: new Set()}],
@@ -70,6 +71,7 @@ export class KeyboardShortcutController {
             ['scanTextAtCaret',                  {scopes: new Set(['web'])}],
             ['toggleOption',                     {scopes: new Set(['popup', 'search']), argument: {template: 'hotkey-argument-setting-path', default: ''}}]
         ]);
+        /* eslint-enable no-multi-spaces */
     }
 
     /** @type {import('./settings-controller.js').SettingsController} */
diff --git a/ext/js/pages/settings/popup-preview-frame.js b/ext/js/pages/settings/popup-preview-frame.js
index e5cc6580..7828a025 100644
--- a/ext/js/pages/settings/popup-preview-frame.js
+++ b/ext/js/pages/settings/popup-preview-frame.js
@@ -57,6 +57,7 @@ export class PopupPreviewFrame {
         /** @type {string} */
         this._targetOrigin = chrome.runtime.getURL('/').replace(/\/$/, '');
 
+        /* eslint-disable no-multi-spaces */
         /** @type {Map<string, (params: import('core').SerializableObjectAny) => void>} */
         this._windowMessageHandlers = new Map(/** @type {[key: string, handler: (params: import('core').SerializableObjectAny) => void][]} */ ([
             ['PopupPreviewFrame.setText',              this._onSetText.bind(this)],
@@ -64,6 +65,7 @@ export class PopupPreviewFrame {
             ['PopupPreviewFrame.setCustomOuterCss',    this._setCustomOuterCss.bind(this)],
             ['PopupPreviewFrame.updateOptionsContext', this._updateOptionsContext.bind(this)]
         ]));
+        /* eslint-enable no-multi-spaces */
     }
 
     /** */
diff --git a/ext/js/pages/settings/profile-conditions-ui.js b/ext/js/pages/settings/profile-conditions-ui.js
index e02d2585..715aeb69 100644
--- a/ext/js/pages/settings/profile-conditions-ui.js
+++ b/ext/js/pages/settings/profile-conditions-ui.js
@@ -50,6 +50,7 @@ export class ProfileConditionsUI extends EventDispatcher {
         const normalizeInteger = this._normalizeInteger.bind(this);
         const validateFlags = this._validateFlags.bind(this);
         const normalizeFlags = this._normalizeFlags.bind(this);
+        /* eslint-disable no-multi-spaces */
         /** @type {Map<import('profile-conditions-ui').DescriptorType, import('profile-conditions-ui').Descriptor>} */
         this._descriptors = new Map([
             [
@@ -105,6 +106,7 @@ export class ProfileConditionsUI extends EventDispatcher {
                 }
             ]
         ]);
+        /* eslint-enable no-multi-spaces */
         /** @type {Set<string>} */
         this._validFlags = new Set([
             'clipboard'
@@ -217,12 +219,12 @@ export class ProfileConditionsUI extends EventDispatcher {
         const info = this._getOperatorDetails(type, operator);
 
         const {
-            displayName=operator,
-            type: type2='string',
-            defaultValue='',
-            resetDefaultOnChange=false,
-            validate=null,
-            normalize=null
+            displayName = operator,
+            type: type2 = 'string',
+            defaultValue = '',
+            resetDefaultOnChange = false,
+            validate = null,
+            normalize = null
         } = (typeof info === 'undefined' ? {} : info);
 
         return {