When no dictionary is enabled, update preview after enable dictionary (…

…#1232)

* Fix preview not update when enable dictionary

* Add dictionaryEnabled event

* Update logic

* align

* Fix lint

ext/js/pages/settings/dictionary-controller.js

@@ -692,6 +692,9 @@ export class DictionaryController {
         }
 
         const hasEnabledDictionary = (enabledDictionaryCountValid > 0);
+        if (hasEnabledDictionary) {
+            this._settingsController.trigger('dictionaryEnabled', {});
+        }
         for (const node of /** @type {NodeListOf<HTMLElement>} */ (this._noDictionariesEnabledWarnings)) {
             node.hidden = hasEnabledDictionary;
         }

ext/js/pages/settings/popup-preview-controller.js

@@ -48,6 +48,7 @@ export class PopupPreviewController {
         this._frame.addEventListener('load', this._onFrameLoad.bind(this), false);
         this._settingsController.on('optionsContextChanged', this._onOptionsContextChange.bind(this));
         this._settingsController.on('optionsChanged', this._onOptionsChanged.bind(this));
+        this._settingsController.on('dictionaryEnabled', this._onOptionsContextChange.bind(this));
         const languageSelect = querySelectorNotNull(document, '#language-select');
         languageSelect.addEventListener(
             /** @type {string} */ ('settingChanged'),
@@ -86,6 +87,11 @@ export class PopupPreviewController {
         this._invoke('updateOptionsContext', {optionsContext});
     }
 
+    /** */
+    _onDictionaryEnabled() {
+        this._invoke('updateSearch', {});
+    }
+
     /**
      * @param {import('settings-controller').EventArgument<'optionsChanged'>} details
      */

ext/js/pages/settings/popup-preview-frame.js

@@ -69,6 +69,7 @@ export class PopupPreviewFrame {
             ['setCustomOuterCss',      this._setCustomOuterCss.bind(this)],
             ['updateOptionsContext',   this._updateOptionsContext.bind(this)],
             ['setLanguageExampleText', this._setLanguageExampleText.bind(this)],
+            ['updateSearch',           this._updateSearch.bind(this)],
         ]);
         /* eslint-enable @stylistic/no-multi-spaces */
     }

types/ext/popup-preview-frame.d.ts

@@ -55,6 +55,10 @@ export type ApiSurface = {
         };
         return: void;
     };
+    updateSearch: {
+        params: Record<string, never>;
+        return: void;
+    };
 };
 
 export type ApiParams<TName extends ApiNames> = BaseApiParams<ApiSurface[TName]>;

types/ext/settings-controller.d.ts

@@ -39,6 +39,7 @@ export type Events = {
     dictionarySettingsReordered: {
         source: DictionaryController;
     };
+    dictionaryEnabled: Record<string, never>;
     scanInputsChanged: {
         source: ScanInputsController | ScanInputsSimpleController;
     };