alex/yomichan
1
Fork 0
Code Releases Activity

fix "url" not being set on kanji cards, add more fields

Browse Source
This commit is contained in:
Alex Yatskov 2017-02-08 20:13:45 -08:00
parent f3fe0994f5
commit 2e3aec9ba1
3 changed files with 15 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ext/bg/js/options-form.js
View File

@ -351,7 +351,7 @@ function populateAnkiFields(element, options) {
const markers = {
'terms': ['audio', 'dictionary', 'expression', 'furigana', 'glossary', 'reading', 'sentence', 'tags', 'url'],
'kanji': ['character', 'dictionary', 'glossary', 'kunyomi', 'onyomi', 'url']
'kanji': ['character', 'dictionary', 'glossary', 'kunyomi', 'onyomi', 'sentence', 'tags', 'url']
}[tabId] || {};
return anki().getModelFieldNames(modelName).then(names => {

6
ext/fg/js/driver.js
View File

@ -167,7 +167,11 @@ class Driver {
if (definitions.length === 0) {
return false;
} else {
definitions.forEach(definition => definition.url = window.location.href);
const sentence = extractSentence(textSource, this.options.anki.sentenceExt);
definitions.forEach(definition => {
definition.url = window.location.href;
definition.sentence = sentence;
});
this.popup.showNextTo(textSource.getRect());
this.popup.showKanjiDefs(definitions, this.options);

10
ext/fg/js/frame.js
View File

@ -49,7 +49,15 @@ class Frame {
$('.kanji-link').click(e => {
e.preventDefault();
findKanji($(e.target).text()).then(kdefs => this.api_showKanjiDefs({options, definitions: kdefs}));
const character = $(e.target).text();
findKanji(character).then(kdefs => {
kdefs.forEach(kdef => {
kdef.url = definitions[0].url;
kdef.sentence = definitions[0].sentence;
});
this.api_showKanjiDefs({options, definitions: kdefs});
});
});
$('.action-play-audio').click(e => {