summaryrefslogtreecommitdiff
path: root/ext
diff options
context:
space:
mode:
authorAlex Yatskov <alex@foosoft.net>2016-05-28 18:27:26 -0700
committerAlex Yatskov <alex@foosoft.net>2016-05-28 18:27:26 -0700
commit7a37c10aedecd1d3fb5b0a8d28068762c4a3b597 (patch)
tree017efb5de44551b6a75616154b7a5424a4a01527 /ext
parentb229295688396160256d2501d494a9ce221eb5fd (diff)
Options page now works as intended
Diffstat (limited to 'ext')
-rw-r--r--ext/bg/js/options-form.js53
1 files changed, 37 insertions, 16 deletions
diff --git a/ext/bg/js/options-form.js b/ext/bg/js/options-form.js
index 0f2fcc6a..5b7107c7 100644
--- a/ext/bg/js/options-form.js
+++ b/ext/bg/js/options-form.js
@@ -21,6 +21,19 @@ function yomichan() {
return chrome.extension.getBackgroundPage().yomichan;
}
+function fieldsToDict(selection) {
+ const result = {};
+ selection.each((index, element) => {
+ result[$(element).data('field')] = $(element).val();
+ });
+
+ return result;
+}
+
+function modelIdToFieldOptKey(id) {
+ return {'anki-vocab-model': 'ankiVocabFields', 'anki-kanji-model': 'ankiKanjiFields'}[id];
+}
+
function formToOptions(section, callback) {
loadOptions((optsOld) => {
const optsNew = $.extend({}, optsOld);
@@ -34,10 +47,12 @@ function formToOptions(section, callback) {
optsNew.enableAnkiConnect = $('#enable-anki-connect').prop('checked');
break;
case 'anki':
- optsNew.ankiVocabDeck = $('#anki-vocab-deck').val();
- optsNew.ankiVocabModel = $('#anki-vocab-model').val();
- optsNew.ankiKanjiDeck = $('#anki-kanji-deck').val();
- optsNew.ankiKanjiModel = $('#anki-kanji-model').val();
+ optsNew.ankiVocabDeck = $('#anki-vocab-deck').val();
+ optsNew.ankiVocabModel = $('#anki-vocab-model').val();
+ optsNew.ankiVocabFields = fieldsToDict($('#vocab .anki-field-value'));
+ optsNew.ankiKanjiDeck = $('#anki-kanji-deck').val();
+ optsNew.ankiKanjiModel = $('#anki-kanji-model').val();
+ optsNew.ankiKanjiFields = fieldsToDict($('#kanji .anki-field-value'));
break;
}
@@ -77,6 +92,7 @@ function populateAnkiFields(element, opts) {
return;
}
+ const optKey = modelIdToFieldOptKey(element.attr('id'));
yomichan().api_getModelFieldNames({modelName, callback: (names) => {
const table = element.closest('.tab-pane').find('.anki-fields');
table.find('tbody').remove();
@@ -85,7 +101,8 @@ function populateAnkiFields(element, opts) {
names.forEach((name) => {
const row = $('<tr>');
row.append($('<td>').text(name));
- row.append($('<input>', {class: 'anki-field-value form-control'}).data('field', name));
+ const value = opts[optKey][name] || '';
+ row.append($('<input>', {class: 'anki-field-value form-control', value}).data('field', name).change(onOptionsAnkiChanged));
body.append(row);
});
@@ -103,6 +120,9 @@ function onOptionsGeneralChanged(e) {
yomichan().setOptions(optsNew);
if (!optsOld.enableAnkiConnect && optsNew.enableAnkiConnect) {
populateAnkiDeckAndModel(optsNew);
+ $('.options-anki').fadeIn();
+ } else if (optsOld.enableAnkiConnect && !optsNew.enableAnkiConnect) {
+ $('.options-anki').fadeOut();
}
});
});
@@ -116,6 +136,16 @@ function onOptionsAnkiChanged(e) {
}
}
+function onAnkiModelChanged(e) {
+ if (e.originalEvent) {
+ formToOptions('anki', (opts) => {
+ opts[modelIdToFieldOptKey($(this).id)] = {};
+ populateAnkiFields($(this), opts);
+ saveOptions(opts, () => yomichan().setOptions(opts));
+ });
+ }
+}
+
$(document).ready(() => {
loadOptions((opts) => {
$('#scan-length').val(opts.scanLength);
@@ -125,17 +155,8 @@ $(document).ready(() => {
$('#enable-anki-connect').prop('checked', opts.enableAnkiConnect);
$('.options-general input').change(onOptionsGeneralChanged);
- $('.options-anki input, .options-anki select').change(onOptionsAnkiChanged);
- $('.anki-model').change((e) => {
- loadOptions((opts) => populateAnkiFields($(e.currentTarget), opts));
- });
- $('#enable-anki-connect').change((e) => {
- if ($(e.currentTarget).prop('checked')) {
- $('.options-anki').fadeIn();
- } else {
- $('.options-anki').fadeOut();
- }
- });
+ $('.anki-deck').change(onOptionsAnkiChanged);
+ $('.anki-model').change(onAnkiModelChanged);
if (opts.enableAnkiConnect) {
populateAnkiDeckAndModel(opts);