alex/yomichan
1
Fork 0
Code Releases Activity

Rename Popup.setVisible to setVisibleOverride

Browse Source
This commit is contained in:
toasted-nutbread 2019-10-05 21:38:13 -04:00
parent bac237336e
commit 2255fadf52
5 changed files with 12 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -527,8 +527,8 @@ Frontend.runtimeMessageHandlers = {
self.updateOptions();
},
popupSetVisible: (self, {visible}) => {
self.popup.setVisible(visible);
popupSetVisibleOverride: (self, {visible}) => {
self.popup.setVisibleOverride(visible);
}
};

6
ext/fg/js/popup-proxy-host.js
View File

@ -41,7 +41,7 @@ class PopupProxyHost {
show: ({id, elementRect, options}) => this.show(id, elementRect, options),
showOrphaned: ({id, elementRect, options}) => this.show(id, elementRect, options),
hide: ({id, changeFocus}) => this.hide(id, changeFocus),
setVisible: ({id, visible}) => this.setVisible(id, visible),
setVisibleOverride: ({id, visible}) => this.setVisibleOverride(id, visible),
containsPoint: ({id, x, y}) => this.containsPoint(id, x, y),
termsShow: ({id, elementRect, writingMode, definitions, options, context}) => this.termsShow(id, elementRect, writingMode, definitions, options, context),
kanjiShow: ({id, elementRect, writingMode, definitions, options, context}) => this.kanjiShow(id, elementRect, writingMode, definitions, options, context),
@ -103,9 +103,9 @@ class PopupProxyHost {
return popup.hide(changeFocus);
}
async setVisible(id, visible) {
async setVisibleOverride(id, visible) {
const popup = this.getPopup(id);
return popup.setVisible(visible);
return popup.setVisibleOverride(visible);
}
async containsPoint(id, x, y) {

4
ext/fg/js/popup-proxy.js
View File

@ -65,9 +65,9 @@ class PopupProxy {
return await this.invokeHostApi('hide', {id: this.id, changeFocus});
}
async setVisible(visible) {
async setVisibleOverride(visible) {
const id = await this.getPopupId();
return await this.invokeHostApi('setVisible', {id, visible});
return await this.invokeHostApi('setVisibleOverride', {id, visible});
}
async containsPoint(x, y) {

2
ext/fg/js/popup.js
View File

@ -238,7 +238,7 @@ class Popup {
return this.isInjected && this.container.style.visibility !== 'hidden';
}
setVisible(visible) {
setVisibleOverride(visible) {
if (visible) {
this.container.style.setProperty('display', '');
} else {

8
ext/mixed/js/display.js
View File

@ -445,7 +445,7 @@ class Display {
async getScreenshot() {
try {
await this.setPopupVisible(false);
await this.setPopupVisibleOverride(false);
await Display.delay(1); // Wait for popup to be hidden.
const {format, quality} = this.options.anki.screenshot;
@ -454,7 +454,7 @@ class Display {
return {dataUrl, format};
} finally {
await this.setPopupVisible(true);
await this.setPopupVisibleOverride(true);
}
}
@ -462,8 +462,8 @@ class Display {
return this.options.general.resultOutputMode === 'merge' ? 0 : -1;
}
setPopupVisible(visible) {
return apiForward('popupSetVisible', {visible});
setPopupVisibleOverride(visible) {
return apiForward('popupSetVisibleOverride', {visible});
}
setSpinnerVisible(visible) {