Merge branch 'master' into firefox-amo

author: Alex Yatskov <alex@foosoft.net> 2017-04-08 18:04:37 -0700
committer: Alex Yatskov <alex@foosoft.net> 2017-04-08 18:04:37 -0700
commit: 1b49f91a9b252447ab791ccef7b26a0b8b6062dd (patch)
tree: 1cee3846da13d317be015a6ec4c244864e8b9247 /ext/bg
parent: bea50cca462be631dadbd2f5bde1bc0a12ad17bf (diff)
parent: d263a93d446096abfc33e47f591dbb71eb7fe928 (diff)
5 files changed, 34 insertions, 30 deletions
diff --git a/ext/bg/guide.html b/ext/bg/guide.html
index 6c38e351..4b01ae7c 100644
--- a/ext/bg/guide.html
+++ b/ext/bg/guide.html
@@ -9,28 +9,24 @@
     <body>
         <div class="container">
             <div class="page-header">
-                <h1>Welcome to Yomichan!</h1>
+                <h1>Yomichan Usage Guide</h1>
             </div>
 
-            <p>Thank you for downloading this extension! I hope that Yomichan will help you on your language learning journey.</p>
+            <p>
+                Read the steps below to get up and running with Yomichan. For complete documentation,
+                visit the <a href="https://foosoft.net/projects/yomichan/" target="_blank">official homepage</a>.
+            </p>
 
-            <div>
-                <h2>Quick Guide</h2>
+            <ol>
+                <li>Click on the <img src="/mixed/img/icon16.png" alt> icon in the browser toolbar to open the Yomichan actions dialog.</li>
+                <li>Click on the <em>monkey wrench</em> icon in the middle to open the options page.</li>
+                <li>Import the dictionaries you wish to use for term and Kanji searches.</li>
+                <li>Hold down <kbd>Shift</kbd> key or the middle mouse button as you move your mouse over text to display definitions.</li>
+                <li>Click on the <img src="/mixed/img/play-audio.png" alt> icon to hear the term pronounced by a native speaker.</li>
+                <li>Click on individual Kanji in the term definition results to view additional information about those characters.</li>
+            </ol>
 
-                <p>
-                    Read the steps below to get up and running with Yomichan. For complete documentation,
-                    visit the <a href="https://foosoft.net/projects/yomichan/" target="_blank">official homepage</a>.
-                </p>
-
-                <ol>
-                    <li>Click on the <img src="/mixed/img/icon16.png" alt> icon in the browser toolbar to open the Yomichan actions dialog.</li>
-                    <li>Click on the <em>monkey wrench</em> icon in the middle to open the options page.</li>
-                    <li>Import the dictionaries you wish to use for term and Kanji searches.</li>
-                    <li>Hold down <kbd>Shift</kbd> key or the middle mouse button as you move your mouse over text to display definitions.</li>
-                    <li>Click on the <img src="/mixed/img/play-audio.png" alt> icon to hear the term pronounced by a native speaker.</li>
-                    <li>Click on individual Kanji in the term definition results to view additional information about those characters.</li>
-                </ol>
-            </div>
+            <p>This startup notification can be turned off on the Yomichan options page.</p>
         </div>
     </body>
 </html>
diff --git a/ext/bg/js/options.js b/ext/bg/js/options.js
index ad8d83d8..49544840 100644
--- a/ext/bg/js/options.js
+++ b/ext/bg/js/options.js
@@ -25,6 +25,7 @@ function formRead() {
     return optionsLoad().then(optionsOld => {
         const optionsNew = $.extend(true, {}, optionsOld);
 
+        optionsNew.general.showGuide = $('#show-usage-guide').prop('checked');
         optionsNew.general.audioSource = $('#audio-playback-source').val();
         optionsNew.general.audioVolume = $('#audio-playback-volume').val();
         optionsNew.general.groupResults = $('#group-terms-results').prop('checked');
@@ -111,6 +112,7 @@ $(document).ready(() => {
     handlebarsRegister();
 
     optionsLoad().then(options => {
+        $('#show-usage-guide').prop('checked', options.general.showGuide);
         $('#audio-playback-source').val(options.general.audioSource);
         $('#audio-playback-volume').val(options.general.audioVolume);
         $('#group-terms-results').prop('checked', options.general.groupResults);
@@ -368,9 +370,6 @@ function ankiFieldsPopulate(element, options) {
         ],
         'kanji': [
             'character',
-            'cloze-body',
-            'cloze-prefix',
-            'cloze-suffix',
             'dictionary',
             'glossary',
             'kunyomi',
diff --git a/ext/bg/js/util.js b/ext/bg/js/util.js
index 05c7ff27..64143ffe 100644
--- a/ext/bg/js/util.js
+++ b/ext/bg/js/util.js
@@ -89,7 +89,8 @@ function optionsSetDefaults(options) {
             showAdvanced: false,
             popupWidth: 400,
             popupHeight: 250,
-            popupOffset: 10
+            popupOffset: 10,
+            showGuide: true
         },
 
         scanning: {
@@ -144,6 +145,9 @@ function optionsVersion(options) {
             } else {
                 options.general.audioSource = 'disabled';
             }
+        },
+        () => {
+            options.general.showGuide = false;
         }
     ];
 
diff --git a/ext/bg/js/yomichan.js b/ext/bg/js/yomichan.js
index feb74b6e..51e05c80 100644
--- a/ext/bg/js/yomichan.js
+++ b/ext/bg/js/yomichan.js
@@ -28,8 +28,8 @@ window.yomichan = new class {
         this.translator.prepare().then(optionsLoad).then(this.optionsSet.bind(this)).then(() => {
             chrome.commands.onCommand.addListener(this.onCommand.bind(this));
             chrome.runtime.onMessage.addListener(this.onMessage.bind(this));
-            if (chrome.runtime.onInstalled) {
-                chrome.runtime.onInstalled.addListener(this.onInstalled.bind(this));
+            if (this.options.general.showGuide) {
+                chrome.tabs.create({url: chrome.extension.getURL('/bg/guide.html')});
             }
         });
     }
@@ -157,12 +157,6 @@ window.yomichan = new class {
         return Promise.resolve(handlebarsRender(template, data));
     }
 
-    onInstalled(details) {
-        if (details.reason === 'install') {
-            chrome.tabs.create({url: chrome.extension.getURL('/bg/guide.html')});
-        }
-    }
-
     onCommand(command) {
         const handlers = {
             search: () => {
diff --git a/ext/bg/options.html b/ext/bg/options.html
index 939227b0..e6c5e8be 100644
--- a/ext/bg/options.html
+++ b/ext/bg/options.html
@@ -26,6 +26,10 @@
                 <h3>General Options</h3>
 
                 <div class="checkbox">
+                    <label><input type="checkbox" id="show-usage-guide"> Show usage guide on startup</label>
+                </div>
+
+                <div class="checkbox">
                     <label><input type="checkbox" id="group-terms-results"> Group term results</label>
                 </div>
 
@@ -193,6 +197,13 @@
                         <input type="text" id="interface-server" class="form-control">
                     </div>
 
+                    <p class="help-block">
+                        Specify the information you would like included in your flashcards in the field editor below.
+                        Please be aware that Anki requires the first field in the model to be unique. It is highly recommended
+                        that you set it to <code>{expression}</code> for term flashcards and <code>{character}</code> for
+                        Kanji flashcards.
+                    </p>
+
                     <div id="anki-format">
                         <ul class="nav nav-tabs">
                             <li class="active"><a href="#terms" data-toggle="tab">Terms</a></li>
author	Alex Yatskov <alex@foosoft.net>	2017-04-08 18:04:37 -0700
committer	Alex Yatskov <alex@foosoft.net>	2017-04-08 18:04:37 -0700
commit	1b49f91a9b252447ab791ccef7b26a0b8b6062dd (patch)
tree	1cee3846da13d317be015a6ec4c244864e8b9247 /ext/bg
parent	bea50cca462be631dadbd2f5bde1bc0a12ad17bf (diff)
parent	d263a93d446096abfc33e47f591dbb71eb7fe928 (diff)