Make dict moving faster and save scroll location

yomidevs · Dec 16, 2024 · 16ace1a · 16ace1a
1 parent 67a509c
commit 16ace1a
Show file tree

Hide file tree

Showing 2 changed files with 34 additions and 2 deletions.
diff --git a/ext/js/pages/settings/dictionary-controller.js b/ext/js/pages/settings/dictionary-controller.js
@@ -494,6 +494,8 @@ export class DictionaryController {
         this._settingsController = settingsController;
         /** @type {import('./modal-controller.js').ModalController} */
         this._modalController = modalController;
+        /** @type {HTMLElement} */
+        this._dictionaryModalBody = querySelectorNotNull(document, '#dictionaries-modal-body');
         /** @type {import('./status-footer.js').StatusFooter} */
         this._statusFooter = statusFooter;
         /** @type {?import('dictionary-importer').Summary[]} */
@@ -643,7 +645,7 @@ export class DictionaryController {
         const event = {source: this};
         this._settingsController.trigger('dictionarySettingsReordered', event);
 
-        await this._updateEntries();
+        this._updateCurrentEntries(options);
     }
 
     /**
@@ -822,6 +824,36 @@ export class DictionaryController {
         }
     }
 
+    /**
+     * @param {import('settings').ProfileOptions} options
+     */
+    _updateCurrentEntries(options) {
+        const dictionariesModalBodyScrollY = this._dictionaryModalBody.scrollTop;
+        const dictionaries = this._dictionaries;
+        if (dictionaries === null) { return; }
+
+        for (const entry of this._dictionaryEntries) {
+            entry.cleanup();
+        }
+
+        /** @type {Map<string, import('dictionary-importer').Summary>} */
+        const dictionaryInfoMap = new Map();
+        for (const dictionary of dictionaries) {
+            dictionaryInfoMap.set(dictionary.title, dictionary);
+        }
+
+        const dictionaryOptionsArray = options.dictionaries;
+        for (let i = 0, ii = dictionaryOptionsArray.length; i < ii; ++i) {
+            const {name} = dictionaryOptionsArray[i];
+            /** @type {import('dictionary-importer').Summary | undefined} */
+            const dictionaryInfo = dictionaryInfoMap.get(name);
+            if (typeof dictionaryInfo === 'undefined') { continue; }
+            const updateDownloadUrl = dictionaryInfo.downloadUrl ?? null;
+            this._createDictionaryEntry(i, dictionaryInfo, updateDownloadUrl);
+        }
+        this._dictionaryModalBody.scroll({top: dictionariesModalBodyScrollY});
+    }
+
     /**
      * @param {import('settings').ProfileOptions} options
      */

diff --git a/ext/templates-modals.html b/ext/templates-modals.html
@@ -11,7 +11,7 @@
                 </div>
             </div>
         </div>
-        <div class="modal-body">
+        <div id="dictionaries-modal-body" class="modal-body">
             <div class="settings-item">
                 <div class="settings-item-inner">
                     <div class="settings-item-left">