alex/yomichan
1
Fork 0
Code Releases Activity

Updating options page

Browse Source
This commit is contained in:
Alex Yatskov 2016-05-01 21:18:28 -07:00
parent 61993db702
commit eaa7291683
4 changed files with 22 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -18,16 +18,18 @@
function optionsToForm(opts) {
$('#scanLength').val(opts.scanLength);
$('#enableAnkiConnect').prop('checked', opts.ankiConnect);
$('#selectMatchedText').prop('checked', opts.selectMatchedText);
$('#loadOnStartup').prop('checked', opts.loadOnStartup);
$('#highlightText').prop('checked', opts.highlightText);
$('#scanLength').val(opts.scanLength);
}
function formToOptions() {
return sanitizeOptions({
scanLength: $('#scanLength').val(),
loadOnStartup: $('#loadOnStartup').prop('checked'),
highlightText: $('#highlightText').prop('checked')
loadOnStartup: $('#loadOnStartup').prop('checked'),
selectMatchedText: $('#selectMatchedText').prop('checked'),
ankiConnect: $('#enableAnkiConnect').prop('checked'),
scanLength: $('#scanLength').val()
});
}

7
ext/bg/js/options.js
View File

@ -19,9 +19,10 @@
function sanitizeOptions(options) {
const defaults = {
scanLength: 20,
loadOnStartup: false,
highlightText: true
loadOnStartup: false,
selectMatchedText: true,
enableAnkiConnect: false,
scanLength: 20
};
for (const key in defaults) {

10
ext/bg/options.html
View File

@ -34,7 +34,15 @@
<div class="form-group">
<div class="col-sm-offset-2 col-sm-10">
<div class="checkbox">
<label><input type="checkbox" id="highlightText" checked> Highlight text</label>
<label><input type="checkbox" id="selectMatchedText" checked> Select matched text</label>
</div>
</div>
</div>
<div class="form-group">
<div class="col-sm-offset-2 col-sm-10">
<div class="checkbox">
<label><input type="checkbox" id="enableAnkiConnect" checked> Enable AnkiConnect</label>
</div>
</div>
</div>

4
ext/fg/js/client.js
View File

@ -126,7 +126,7 @@ class Client {
showPopup(range, content) {
this.popup.showNextTo(range.getRect(), content);
if (this.options.highlightText) {
if (this.options.selectMatchedText) {
range.select();
}
@ -136,7 +136,7 @@ class Client {
hidePopup() {
this.popup.hide();
if (this.options.highlightText && this.lastRange !== null) {
if (this.options.selectMatchedText && this.lastRange !== null) {
this.lastRange.deselect();
}