fix lint issues

yomidevs · Jul 30, 2024 · a8d1601 · a8d1601
1 parent f457ff3
commit a8d1601
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/ext/js/pages/settings/dictionary-controller.js b/ext/js/pages/settings/dictionary-controller.js
@@ -183,6 +183,14 @@ class DictionaryEntry {
         }
     }
 
+    /**
+     * @param {string} alias
+     */
+    updateAliasSettings(alias) {
+        this._aliasNode.textContent = alias;
+        this._aliasNode.dispatchEvent(new CustomEvent('change', {bubbles: true}));
+    }
+
     /**
      * @param {import('dom-data-binder').SettingChangedEvent} e
      */
@@ -916,10 +924,9 @@ export class DictionaryController {
         /** @type {HTMLInputElement} */
         const input = querySelectorNotNull(modal.node, '#dictionary-alias-input');
         const inputValue = input.value.trim();
-        if (!inputValue) return;
-        const aliasNode = this._dictionaryEntries[indexNumber]._aliasNode;
-        aliasNode.textContent = inputValue;
-        aliasNode.dispatchEvent(new Event('change', {bubbles: true}));
+        if (inputValue) {
+            this._dictionaryEntries[indexNumber].updateAliasSettings(inputValue);
+        }
     }
 
     /**

diff --git a/ext/settings.html b/ext/settings.html
@@ -2835,7 +2835,6 @@ <h5>or click here to upload</h5>
     </div>
 </div></div>
 
-
 <!-- Custom CSS modal -->
 <div id="custom-css-modal" class="modal modal-left" tabindex="-1" role="dialog" hidden>
     <div class="modal-content-container">