alex/yomichan
1
Fork 0
Code Releases Activity

Don't trigger a change event for the initial content of the clipboard (#1321)

Browse Source
This commit is contained in:
toasted-nutbread 2021-01-26 20:12:04 -05:00 committed by GitHub
parent 77b7bdb4ce
commit 97bb05147e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
ext/bg/js/clipboard-monitor.js
View File

@ -32,6 +32,7 @@ class ClipboardMonitor extends EventDispatcher {
// The token below is used as a unique identifier to ensure that a new clipboard monitor
// hasn't been started during the await call. The check below the await call
// will exit early if the reference has changed.
let canChange = false;
const token = {};
const intervalCallback = async () => {
this._timerId = null;
@ -50,11 +51,12 @@ class ClipboardMonitor extends EventDispatcher {
text !== this._previousText
) {
this._previousText = text;
if (this._japaneseUtil.isStringPartiallyJapanese(text)) {
if (canChange && this._japaneseUtil.isStringPartiallyJapanese(text)) {
this.trigger('change', {text});
}
}
canChange = true;
this._timerId = setTimeout(intervalCallback, this._interval);
};
@ -65,6 +67,7 @@ class ClipboardMonitor extends EventDispatcher {
stop() {
this._timerToken = null;
this._previousText = null;
if (this._timerId !== null) {
clearTimeout(this._timerId);
this._timerId = null;