Helper text when no dictionary enabled

yomidevs · Jul 13, 2024 · 5b90d8e · 5b90d8e
1 parent 165ed8d
commit 5b90d8e
Show file tree

Hide file tree

Showing 3 changed files with 28 additions and 1 deletion.
diff --git a/ext/js/display/display.js b/ext/js/display/display.js
@@ -1296,6 +1296,7 @@ export class Display extends EventDispatcher {
     async _setContentTermsOrKanji(type, urlSearchParams, token) {
         const lookup = (urlSearchParams.get('lookup') !== 'false');
         const wildcardsEnabled = (urlSearchParams.get('wildcards') !== 'off');
+        const hasEnabledDictionaries = this._options ? this._options.dictionaries.some(({enabled}) => enabled) : false;
 
         // Set query
         let query = urlSearchParams.get('query');
@@ -1367,7 +1368,8 @@ export class Display extends EventDispatcher {
         this._dictionaryEntries = dictionaryEntries;
 
         this._updateNavigationAuto();
-        this._setNoContentVisible(dictionaryEntries.length === 0 && lookup);
+        this._setNoContentVisible(hasEnabledDictionaries && dictionaryEntries.length === 0 && lookup);
+        this._setNoDictionariesVisible(!hasEnabledDictionaries);
 
         const container = this._container;
         container.textContent = '';
@@ -1424,6 +1426,7 @@ export class Display extends EventDispatcher {
 
         this._updateNavigation(false, false);
         this._setNoContentVisible(false);
+        this._setNoDictionariesVisible(false);
         this._setQuery('', '', 0);
 
         this._triggerContentUpdateStart();
@@ -1452,6 +1455,18 @@ export class Display extends EventDispatcher {
         }
     }
 
+    /**
+     * @param {boolean} visible
+     */
+    _setNoDictionariesVisible(visible) {
+        /** @type {?HTMLElement} */
+        const noDictionaries = document.querySelector('#no-dictionaries');
+
+        if (noDictionaries !== null) {
+            noDictionaries.hidden = !visible;
+        }
+    }
+
     /**
      * @param {string} query
      * @param {string} fullQuery

diff --git a/ext/popup.html b/ext/popup.html
@@ -45,6 +45,12 @@
                         </div>
                     </div>
 
+                    <div id="no-dictionaries" hidden>
+                        <div class="entry">
+                            <p>No dictionaries have been installed or enabled yet.</p>
+                            <p><a href="#">Go to Dictionaries settings.</a></p>
+                        </div>
+                    </div>
                     <div id="error-extension-unloaded" hidden>
                         <div class="entry">
                             <h1>Yomitan Updated!</h1>

diff --git a/ext/search.html b/ext/search.html
@@ -80,6 +80,12 @@ <h1>Yomitan Search</h1>
                         </div>
                     </div>
 
+                    <div id="no-dictionaries" hidden>
+                        <div class="entry">
+                            <p>No dictionaries have been installed or enabled yet.</p>
+                            <p><a href="/settings.html#!dictionaries" target="_blank">Go to Dictionaries settings.</a></p>
+                        </div>
+                    </div>
                 </div>
             </div>
         </div>