diff options
author | toasted-nutbread <toasted-nutbread@users.noreply.github.com> | 2021-02-14 17:52:01 -0500 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-02-14 17:52:01 -0500 |
commit | 9279ced68660610764931da681f22c8b71bf1b6e (patch) | |
tree | 2a4d015cf57adfe74af72615bad19c9308388e99 /ext/js/app | |
parent | 286534e648af350d24fbf3c7892a7ec81aaeb4bd (diff) |
Log refactoring (#1393)
* Create Logger class and log instance
* Replace yomichan.logWarning with log.warn
* Replace yomichan.logError with log.error
* Replace yomichan.log with log.log
* Update the Yomichan class to use the global log object
* Update lint rules
Diffstat (limited to 'ext/js/app')
-rw-r--r-- | ext/js/app/content-script-main.js | 2 | ||||
-rw-r--r-- | ext/js/app/frontend.js | 4 | ||||
-rw-r--r-- | ext/js/app/popup-proxy.js | 2 |
3 files changed, 4 insertions, 4 deletions
diff --git a/ext/js/app/content-script-main.js b/ext/js/app/content-script-main.js index a09e52ea..1c0c6d49 100644 --- a/ext/js/app/content-script-main.js +++ b/ext/js/app/content-script-main.js @@ -52,6 +52,6 @@ yomichan.ready(); } catch (e) { - yomichan.logError(e); + log.error(e); } })(); diff --git a/ext/js/app/frontend.js b/ext/js/app/frontend.js index 74cc63d2..02707f41 100644 --- a/ext/js/app/frontend.js +++ b/ext/js/app/frontend.js @@ -276,7 +276,7 @@ class Frontend { this._showExtensionUnloaded(textSource); } } else { - yomichan.logError(error); + log.error(error); } } if (type !== null) { this._stopClearSelectionDelayed(); @@ -563,7 +563,7 @@ class Frontend { ); this._lastShowPromise.catch((error) => { if (yomichan.isExtensionUnloaded) { return; } - yomichan.logError(error); + log.error(error); }); return this._lastShowPromise; } diff --git a/ext/js/app/popup-proxy.js b/ext/js/app/popup-proxy.js index 19856e3f..63ecc254 100644 --- a/ext/js/app/popup-proxy.js +++ b/ext/js/app/popup-proxy.js @@ -201,7 +201,7 @@ class PopupProxy extends EventDispatcher { } this._frameOffsetUpdatedAt = now; } catch (e) { - yomichan.logError(e); + log.error(e); } finally { this._frameOffsetPromise = null; } |