From 0b602925ab54932adbc9910337b5ee5993c44326 Mon Sep 17 00:00:00 2001
From: Alex Yatskov <alex@foosoft.net>
Date: Sat, 28 Oct 2017 08:11:33 -0700
Subject: cleanup

---
 ext/bg/settings.html    | 12 ++++++------
 ext/mixed/js/display.js |  6 ++++--
 2 files changed, 10 insertions(+), 8 deletions(-)

(limited to 'ext')

diff --git a/ext/bg/settings.html b/ext/bg/settings.html
index f430d5e2..12556065 100644
--- a/ext/bg/settings.html
+++ b/ext/bg/settings.html
@@ -55,7 +55,7 @@
                     <label for="result-output-mode">Result grouping</label>
                     <select class="form-control" id="result-output-mode">
                         <option value="group">Group results by term-reading pairs</option>
-                        <option value="merge">Group results by main dictionary entry ID (experimental)</option>
+                        <option value="merge">Group results by main dictionary entry</option>
                         <option value="split">Split definitions to their own results</option>
                     </select>
                 </div>
@@ -140,11 +140,6 @@
                     <h3>Dictionaries</h3>
                 </div>
 
-                <div class="form-group options-merge">
-                    <label for="main-dictionary">Main dictionary</label>
-                    <select class="form-control" id="main-dictionary"></select>
-                </div>
-
                 <p class="help-block">
                     Yomichan can import and use a variety of dictionary formats. Unneeded dictionaries can be disabled,
                     or you can simply <a href="#" id="dict-purge-link">purge the database</a> to delete everything.
@@ -160,6 +155,11 @@
 
                 <div id="dict-groups"></div>
 
+                <div class="form-group options-merge">
+                    <label for="main-dictionary">Main dictionary</label>
+                    <select class="form-control" id="main-dictionary"></select>
+                </div>
+
                 <div id="dict-import-progress">
                     Dictionary data is being imported, please be patient...
                     <div class="progress">
diff --git a/ext/mixed/js/display.js b/ext/mixed/js/display.js
index 41fe85eb..5d3c4f2e 100644
--- a/ext/mixed/js/display.js
+++ b/ext/mixed/js/display.js
@@ -185,7 +185,8 @@ class Display {
             80: /* p */ () => {
                 if (e.altKey) {
                     if ($('.entry').eq(this.index).data('type') === 'term') {
-                        this.audioPlay(this.definitions[this.index], this.options.general.resultOutputMode === 'merge' ? 0 : -1);
+                        const expressionIndex = this.options.general.resultOutputMode === 'merge' ? 0 : -1;
+                        this.audioPlay(this.definitions[this.index], expressionIndex);
                     }
 
                     return true;
@@ -387,7 +388,8 @@ class Display {
         try {
             this.spinner.show();
 
-            let url = await apiAudioGetUrl(expressionIndex === -1 ? definition : definition.expressions[expressionIndex], this.options.general.audioSource);
+            const expression = expressionIndex === -1 ? definition : definition.expressions[expressionIndex];
+            let url = await apiAudioGetUrl(expression, this.options.general.audioSource);
             if (!url) {
                 url = '/mixed/mp3/button.mp3';
             }
-- 
cgit v1.2.3