alex/yomichan
1
Fork 0
Code Releases Activity

Update comment on getClipboardText

Browse Source
This commit is contained in:
toasted-nutbread 2019-12-15 13:27:26 -05:00
parent dec7f6cabb
commit 024f969bfd
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
ext/bg/js/search.js
View File

@ -286,8 +286,10 @@ class DisplaySearch extends Display {
async getClipboardText() {
/*
Notes:
apiClipboardGet doesn't work on firefox because document.execCommand('paste') requires
user interaction. Therefore, navigator.clipboard.readText() is used.
apiClipboardGet doesn't work on Firefox because document.execCommand('paste')
results in an empty string on the web extension background page.
This may be a bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1603985
Therefore, navigator.clipboard.readText() is used on Firefox.
navigator.clipboard.readText() can't be used in Chrome for two reasons:
* Requires page to be focused, else it rejects with an exception.