More cleanup
This commit is contained in:
parent
18fc8d9865
commit
8416c9d633
@ -158,9 +158,7 @@ class Client {
|
||||
}
|
||||
|
||||
api_addNote({index, mode}) {
|
||||
const state = {};
|
||||
state[mode] = false;
|
||||
|
||||
const state = {[mode]: false};
|
||||
bgAddDefinition(this.definitions[index], mode).then((success) => {
|
||||
if (success) {
|
||||
this.popup.sendMessage('setActionState', {index, state, sequence: this.sequence});
|
||||
|
@ -18,7 +18,7 @@
|
||||
|
||||
|
||||
function registerKanjiLinks() {
|
||||
for (let link of [].slice.call(document.getElementsByClassName('kanji-link'))) {
|
||||
for (const link of Array.from(document.getElementsByClassName('kanji-link'))) {
|
||||
link.addEventListener('click', (e) => {
|
||||
e.preventDefault();
|
||||
window.parent.postMessage({action: 'displayKanji', params: e.target.innerHTML}, '*');
|
||||
@ -27,7 +27,7 @@ function registerKanjiLinks() {
|
||||
}
|
||||
|
||||
function registerAddNoteLinks() {
|
||||
for (let link of [].slice.call(document.getElementsByClassName('action-add-note'))) {
|
||||
for (const link of Array.from(document.getElementsByClassName('action-add-note'))) {
|
||||
link.addEventListener('click', (e) => {
|
||||
e.preventDefault();
|
||||
const ds = e.currentTarget.dataset;
|
||||
@ -37,7 +37,7 @@ function registerAddNoteLinks() {
|
||||
}
|
||||
|
||||
function registerAudioLinks() {
|
||||
for (let link of [].slice.call(document.getElementsByClassName('action-play-audio'))) {
|
||||
for (const link of Array.from(document.getElementsByClassName('action-play-audio'))) {
|
||||
link.addEventListener('click', (e) => {
|
||||
e.preventDefault();
|
||||
const ds = e.currentTarget.dataset;
|
||||
@ -60,7 +60,7 @@ function onMessage(e) {
|
||||
}
|
||||
|
||||
function api_setActionState({index, state, sequence}) {
|
||||
for (let mode in state) {
|
||||
for (const mode in state) {
|
||||
const matches = document.querySelectorAll(`.action-bar[data-sequence="${sequence}"] .action-add-note[data-index="${index}"][data-mode="${mode}"]`);
|
||||
if (matches.length === 0) {
|
||||
return;
|
||||
|
@ -19,15 +19,15 @@
|
||||
|
||||
class Popup {
|
||||
constructor() {
|
||||
this.popup = null;
|
||||
this.popup = null;
|
||||
this.offset = 10;
|
||||
}
|
||||
|
||||
showAt(pos, content) {
|
||||
this.inject();
|
||||
|
||||
this.popup.style.left = pos.x + 'px';
|
||||
this.popup.style.top = pos.y + 'px';
|
||||
this.popup.style.left = pos.x + 'px';
|
||||
this.popup.style.top = pos.y + 'px';
|
||||
this.popup.style.visibility = 'visible';
|
||||
|
||||
this.setContent(content);
|
||||
|
@ -71,6 +71,6 @@ class TextSourceElement {
|
||||
}
|
||||
|
||||
equals(other) {
|
||||
return other.element && other.textRaw() == this.textRaw();
|
||||
return other.element && other.textRaw() === this.textRaw();
|
||||
}
|
||||
}
|
||||
|
@ -81,7 +81,7 @@ class TextSourceRange {
|
||||
}
|
||||
|
||||
static seekForward(node, length) {
|
||||
const state = {node, offset: 0, length};
|
||||
const state = {node, length, offset: 0};
|
||||
if (!TextSourceRange.seekForwardHelper(node, state)) {
|
||||
return state;
|
||||
}
|
||||
@ -115,7 +115,7 @@ class TextSourceRange {
|
||||
}
|
||||
|
||||
static seekBackward(node, length) {
|
||||
const state = {node, offset: node.length, length};
|
||||
const state = {node, length, offset: node.length};
|
||||
if (!TextSourceRange.seekBackwardHelper(node, state)) {
|
||||
return state;
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user