fix: make copyListener async

aminya · aminya · commit ceff0e2971f3 · 2021-02-02T12:17:05.000-06:00
diff --git a/lib/datatip-manager.ts b/lib/datatip-manager.ts
@@ -535,11 +535,11 @@ export class DataTipManager {
 
 // TODO we should not need this
 /** A manual copy listener */
-function copyListener(event: KeyboardEvent) {
+async function copyListener(event: KeyboardEvent) {
   event.preventDefault()
   if (event.ctrlKey && event.key === "c") {
     const text = document.getSelection()?.toString() ?? ""
-    navigator.clipboard.writeText(text).catch(() => {})
+    await navigator.clipboard.writeText(text)
   }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -535,11 +535,11 @@ export class DataTipManager {`
`535`	`535`
`536`	`536`	`// TODO we should not need this`
`537`	`537`	`/** A manual copy listener */`
`538`		`-function copyListener(event: KeyboardEvent) {`
	`538`	`+async function copyListener(event: KeyboardEvent) {`
`539`	`539`	`event.preventDefault()`
`540`	`540`	`if (event.ctrlKey && event.key === "c") {`
`541`	`541`	`const text = document.getSelection()?.toString() ?? ""`
`542`		`- navigator.clipboard.writeText(text).catch(() => {})`
	`542`	`+ await navigator.clipboard.writeText(text)`
`543`	`543`	`}`
`544`	`544`	`}`
`545`	`545`