summaryrefslogtreecommitdiff
path: root/ext/js/pages/settings/collapsible-dictionary-controller.js
diff options
context:
space:
mode:
authortoasted-nutbread <toasted-nutbread@users.noreply.github.com>2021-03-31 18:53:08 -0400
committerGitHub <noreply@github.com>2021-03-31 18:53:08 -0400
commitf4af3f31efb41a1379868c6f128e7a03ee05fd2b (patch)
treefb49aec6d6e60fe649dc17ab5bbd805b59a49425 /ext/js/pages/settings/collapsible-dictionary-controller.js
parentbdec71976a9478beaf74d8299aa1ca6a95c2ce0c (diff)
Make secondary dictionary option controllers more consistent (#1577)
Diffstat (limited to 'ext/js/pages/settings/collapsible-dictionary-controller.js')
-rw-r--r--ext/js/pages/settings/collapsible-dictionary-controller.js15
1 files changed, 4 insertions, 11 deletions
diff --git a/ext/js/pages/settings/collapsible-dictionary-controller.js b/ext/js/pages/settings/collapsible-dictionary-controller.js
index a3a1df62..37793c6f 100644
--- a/ext/js/pages/settings/collapsible-dictionary-controller.js
+++ b/ext/js/pages/settings/collapsible-dictionary-controller.js
@@ -24,17 +24,16 @@ class CollapsibleDictionaryController {
this._settingsController = settingsController;
this._getDictionaryInfoToken = null;
this._dictionaryInfoMap = new Map();
+ this._eventListeners = new EventListenerCollection();
this._container = null;
this._selects = [];
this._allSelect = null;
- this._eventListeners = new EventListenerCollection();
}
async prepare() {
this._container = document.querySelector('#collapsible-dictionary-list');
await this._onDatabaseUpdated();
- await this._updateOptions();
yomichan.on('databaseUpdated', this._onDatabaseUpdated.bind(this));
this._settingsController.on('optionsChanged', this._onOptionsChanged.bind(this));
@@ -54,20 +53,19 @@ class CollapsibleDictionaryController {
this._dictionaryInfoMap.set(entry.title, entry);
}
- await this._updateOptions();
+ const options = await this._settingsController.getOptions();
+ this._onOptionsChanged({options});
}
_onOptionsChanged({options}) {
this._eventListeners.removeAllEventListeners();
this._selects = [];
- const {dictionaries} = options;
-
const fragment = document.createDocumentFragment();
this._setupAllSelect(fragment, options);
- for (const dictionary of Object.keys(dictionaries)) {
+ for (const dictionary of Object.keys(options.dictionaries)) {
const dictionaryInfo = this._dictionaryInfoMap.get(dictionary);
if (typeof dictionaryInfo === 'undefined') { continue; }
@@ -120,11 +118,6 @@ class CollapsibleDictionaryController {
return node.querySelector('.definitions-collapsible');
}
- async _updateOptions() {
- const options = await this._settingsController.getOptions();
- this._onOptionsChanged({options});
- }
-
async _updateAllSelectFresh() {
this._updateAllSelect(await this._settingsController.getOptions());
}