Wrap with try/catch & drop the uneeded .then()

umbraco · Jan 15, 2025 · ac50a35 · ac50a35
1 parent 2a8db77
commit ac50a35
Showing 1 changed file with 14 additions and 16 deletions.
diff --git a/packages/uui-text-copy/lib/uui-text-copy.element.ts b/packages/uui-text-copy/lib/uui-text-copy.element.ts
@@ -127,22 +127,20 @@ export class UUITextCopyElement extends LabelMixin('', LitElement) {
       this.#valueToCopy = beforeCopyEv.detail.text;
     }
 
-    await navigator.clipboard
-      .writeText(this.#valueToCopy)
-      .then(() => {
-        this.dispatchEvent(
-          new UUITextCopyEvent(UUITextCopyEvent.COPIED, {
-            detail: { text: this.#valueToCopy },
-          }),
-        );
-        setTimeout(() => {
-          button.state = 'success';
-        }, this.animationStateDelay);
-      })
-      .catch(err => {
-        button.state = 'failed';
-        console.error('Error copying to clipboard', err);
-      });
+    try {
+      await navigator.clipboard.writeText(this.#valueToCopy);
+      this.dispatchEvent(
+        new UUITextCopyEvent(UUITextCopyEvent.COPIED, {
+          detail: { text: this.#valueToCopy },
+        }),
+      );
+      setTimeout(() => {
+        button.state = 'success';
+      }, this.animationStateDelay);
+    } catch (err) {
+      button.state = 'failed';
+      console.error('Error copying to clipboard', err);
+    }
   };
 
   render() {