aboutsummaryrefslogtreecommitdiff
path: root/ext
diff options
context:
space:
mode:
authorAlex Yatskov <alex@foosoft.net>2017-01-09 09:30:56 -0800
committerAlex Yatskov <alex@foosoft.net>2017-01-09 09:30:56 -0800
commit89070c609645455ac307bd0de918f2b76726dd3a (patch)
tree266b2586a03968404ef95ac80d284aa5aa0026a7 /ext
parent60e23ab42c86fbc74a6e16d6fd29fcc6761bfb36 (diff)
cleanup
Diffstat (limited to 'ext')
-rw-r--r--ext/fg/js/driver.js10
-rw-r--r--ext/fg/js/frame.js32
-rw-r--r--ext/fg/js/popup.js4
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});
}