From 89070c609645455ac307bd0de918f2b76726dd3a Mon Sep 17 00:00:00 2001
From: Alex Yatskov <alex@foosoft.net>
Date: Mon, 9 Jan 2017 09:30:56 -0800
Subject: [PATCH] cleanup

---
 ext/fg/js/driver.js | 10 ++--------
 ext/fg/js/frame.js  | 32 --------------------------------
 ext/fg/js/popup.js  |  4 ----
 3 files changed, 2 insertions(+), 44 deletions(-)

diff --git a/ext/fg/js/driver.js b/ext/fg/js/driver.js
index 12003d55..ef7db481 100644
--- a/ext/fg/js/driver.js
+++ b/ext/fg/js/driver.js
@@ -146,13 +146,7 @@ class Driver {
     searchTerms(textSource) {
         textSource.setEndOffset(this.options.scanLength);
 
-        let findFunc = findTerm;
-        let showFunc = this.popup.showTermDefs.bind(this.popup);
-        if (this.options.groupTermResults) {
-            findFunc = findTermGrouped;
-            showFunc = this.popup.showTermGroupedDefs.bind(this.popup);
-        }
-
+        const findFunc = this.options.groupTermResults ? findTermGrouped : findTerm;
         return findFunc(textSource.text()).then(({definitions, length}) => {
             if (definitions.length === 0) {
                 return false;
@@ -166,7 +160,7 @@ class Driver {
                 });
 
                 this.popup.showNextTo(textSource.getRect());
-                showFunc(definitions, this.options);
+                this.popup.showTermDefs(definitions, this.options);
                 this.lastTextSource = textSource;
                 if (this.options.selectMatchedText) {
                     textSource.select();
diff --git a/ext/fg/js/frame.js b/ext/fg/js/frame.js
index 2cf32e3e..4295dbb3 100644
--- a/ext/fg/js/frame.js
+++ b/ext/fg/js/frame.js
@@ -62,38 +62,6 @@ class Frame {
         });
     }
 
-    api_showTermGroupedDefs({definitions, options}) {
-        const sequence = ++this.sequence;
-        const context = {
-            definitions,
-            grouped: options.groupTermResults,
-            addable: options.ankiMethod !== 'disabled',
-            playback: options.enableAudioPlayback
-        };
-
-        this.definitions = definitions;
-        this.showSpinner(false);
-        window.scrollTo(0, 0);
-
-        renderText(context, 'term-list.html').then(content => {
-            $('.content').html(content);
-            $('.action-add-note').click(this.onAddNote.bind(this));
-
-            $('.kanji-link').click(e => {
-                e.preventDefault();
-                findKanji($(e.target).text()).then(kdefs => this.api_showKanjiDefs({options, definitions: kdefs}));
-            });
-
-            $('.action-play-audio').click(e => {
-                e.preventDefault();
-                const index = $(e.currentTarget).data('index');
-                this.playAudio(this.definitions[index]);
-            });
-
-            this.updateAddNoteButtons(['term_kanji', 'term_kana'], sequence);
-        });
-    }
-
     api_showKanjiDefs({definitions, options}) {
         const sequence = ++this.sequence;
         const context = {
diff --git a/ext/fg/js/popup.js b/ext/fg/js/popup.js
index d2c78081..21f4a9d7 100644
--- a/ext/fg/js/popup.js
+++ b/ext/fg/js/popup.js
@@ -74,10 +74,6 @@ class Popup {
         this.invokeApi('showTermDefs', {definitions, options});
     }
 
-    showTermGroupedDefs(definitions, options) {
-        this.invokeApi('showTermGroupedDefs', {definitions, options});
-    }
-
     showKanjiDefs(definitions, options) {
         this.invokeApi('showKanjiDefs', {definitions, options});
     }