alex/yomichan
1
Fork 0
Code Releases Activity

Rename optionsUpdate event to optionsUpdated

Browse Source 
Past tense better indicates that the options were changed, but no data
is being included as part of the event. It is also more consistent with
the other event names the yomichan object currently provides.
This commit is contained in:
toasted-nutbread 2020-02-14 22:34:44 -05:00
parent 89cb98e4d1
commit faf15c08aa
5 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ext/bg/js/backend.js
View File

@ -88,7 +88,7 @@ class Backend {
const callback = () => this.checkLastError(chrome.runtime.lastError);
chrome.tabs.query({}, (tabs) => {
for (const tab of tabs) {
chrome.tabs.sendMessage(tab.id, {action: 'optionsUpdate', params: {source}}, callback);
chrome.tabs.sendMessage(tab.id, {action: 'optionsUpdated', params: {source}}, callback);
}
});
}

4
ext/bg/js/settings/main.js
View File

@ -242,7 +242,7 @@ async function settingsSaveOptions() {
await apiOptionsSave(source);
}
async function onOptionsUpdate({source}) {
async function onOptionsUpdated({source}) {
const thisSource = await settingsGetSource();
if (source === thisSource) { return; }
@ -274,7 +274,7 @@ async function onReady() {
storageInfoInitialize();
yomichan.on('optionsUpdate', onOptionsUpdate);
yomichan.on('optionsUpdated', onOptionsUpdated);
}
$(document).ready(() => onReady());

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

@ -56,7 +56,7 @@ class Frontend extends TextScanner {
}
yomichan.on('orphaned', () => this.onOrphaned());
yomichan.on('optionsUpdate', () => this.updateOptions());
yomichan.on('optionsUpdated', () => this.updateOptions());
yomichan.on('zoomChanged', (e) => this.onZoomChanged(e));
chrome.runtime.onMessage.addListener(this.onRuntimeMessage.bind(this));

6
ext/mixed/js/core.js
View File

@ -248,7 +248,7 @@ const yomichan = (() => {
this._messageHandlers = new Map([
['getUrl', this._onMessageGetUrl.bind(this)],
['optionsUpdate', this._onMessageOptionsUpdate.bind(this)],
['optionsUpdated', this._onMessageOptionsUpdated.bind(this)],
['zoomChanged', this._onMessageZoomChanged.bind(this)]
]);
@ -276,8 +276,8 @@ const yomichan = (() => {
return {url: window.location.href};
}
_onMessageOptionsUpdate({source}) {
this.trigger('optionsUpdate', {source});
_onMessageOptionsUpdated({source}) {
this.trigger('optionsUpdated', {source});
}
_onMessageZoomChanged({oldZoomFactor, newZoomFactor}) {

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

@ -52,7 +52,7 @@ class Display {
const displayGeneratorPromise = this.displayGenerator.prepare();
const updateOptionsPromise = this.updateOptions(options);
await Promise.all([displayGeneratorPromise, updateOptionsPromise]);
yomichan.on('optionsUpdate', () => this.updateOptions(null));
yomichan.on('optionsUpdated', () => this.updateOptions(null));
}
onError(_error) {