Hook up sentence search extent to options

This commit is contained in:
Alex Yatskov 2016-07-25 20:28:56 -07:00
parent 2b2e6cba3f
commit 80fedc4f7a
4 changed files with 12 additions and 5 deletions

View File

@ -49,6 +49,7 @@ function formToOptions(section, callback) {
break;
case 'anki':
optsNew.ankiCardTags = $('#anki-card-tags').val().split(/[,; ]+/);
optsNew.sentenceExtent = parseInt($('#sentence-extent').val());
optsNew.ankiVocabDeck = $('#anki-vocab-deck').val();
optsNew.ankiVocabModel = $('#anki-vocab-model').val();
optsNew.ankiVocabFields = fieldsToDict($('#vocab .anki-field-value'));
@ -174,6 +175,7 @@ $(document).ready(() => {
$('#enable-anki-connect').prop('checked', opts.enableAnkiConnect);
$('#anki-card-tags').val(opts.ankiCardTags.join(' '));
$('#sentence-extent').val(opts.sentenceExtent);
$('.options-general input').change(onOptionsGeneralChanged);
$('.options-anki input').change(onOptionsAnkiChanged);

View File

@ -26,6 +26,7 @@ function sanitizeOptions(options) {
enableAudioPlayback: true,
enableAnkiConnect: false,
ankiCardTags: ['yomichan'],
sentenceExtent: 200,
ankiVocabDeck: '',
ankiVocabModel: '',
ankiVocabFields: {},

View File

@ -12,9 +12,9 @@
.tab-content {
border-bottom: 1px #ddd solid;
border-left: 1px #ddd solid;
border-right: 1px #ddd solid;
padding: 10px;
border-left: 1px #ddd solid;
border-right: 1px #ddd solid;
padding: 10px;
}
.error-dlg {
@ -92,6 +92,11 @@
<div class="col-sm-10"><input type="text" id="anki-card-tags" class="form-control"></div>
</div>
<div class="form-group">
<label for="sentence-extent" class="control-label col-sm-2">Sentence extent</label>
<div class="col-sm-10"><input type="number" min="1" id="sentence-extent" class="form-control"></div>
</div>
<ul class="nav nav-tabs col-sm-offset-2 col-sm-10">
<li class="active"><a href="#vocab" data-toggle="tab">Vocabulary</a></li>
<li><a href="#kanji" data-toggle="tab">Kanji</a></li>

View File

@ -25,7 +25,6 @@ class Client {
this.lastTextSource = null;
this.activateKey = 16;
this.activateBtn = 2;
this.sentenceExtent = 200;
this.enabled = false;
this.options = {};
this.definitions = null;
@ -97,7 +96,7 @@ class Client {
} else {
textSource.setEndOffset(length);
const sentence = Client.extractSentence(textSource, this.sentenceExtent);
const sentence = Client.extractSentence(textSource, this.opts.sentenceExtent);
definitions.forEach((definition) => {
definition.url = window.location.href;
definition.sentence = sentence;