refactor: remove clipboardhelper dependency

Use Clipboard API to copy the stacktrace to clipboard
FlowingCode · Dec 2, 2022 · 2b9ea50 · 2b9ea50
1 parent 9fac202
commit 2b9ea50
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 11 deletions.
diff --git a/pom.xml b/pom.xml
@@ -129,12 +129,7 @@
 			<groupId>javax.annotation</groupId>
 			<artifactId>javax.annotation-api</artifactId>
 			<version>1.3.2</version>
-		</dependency>
-		<dependency>
-		   <groupId>org.vaadin.olli</groupId>
-		   <artifactId>clipboardhelper</artifactId>
-		   <version>1.2.0</version>
-		</dependency>		
+		</dependency>	
     </dependencies>
 
     <build>

diff --git a/src/main/java/com/flowingcode/vaadin/addons/errorwindow/ErrorWindow.java b/src/main/java/com/flowingcode/vaadin/addons/errorwindow/ErrorWindow.java
@@ -23,6 +23,7 @@
 
 import com.vaadin.flow.component.Component;
 import com.vaadin.flow.component.Html;
+import com.vaadin.flow.component.UI;
 import com.vaadin.flow.component.button.Button;
 import com.vaadin.flow.component.button.ButtonVariant;
 import com.vaadin.flow.component.dependency.CssImport;
@@ -39,7 +40,6 @@
 import java.util.stream.Stream;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
-import org.vaadin.olli.ClipboardHelper;
 
 /**
  * Component to visualize an error, caused by an exception, as a modal sub-window. <br>
@@ -177,10 +177,10 @@ private VerticalLayout createMainLayout() {
       // copy details to clipboard button
       Button clipboarButton = new Button(i18n.getClipboard());
       clipboarButton.addThemeVariants(ButtonVariant.LUMO_TERTIARY);
-      ClipboardHelper clipboardHelper = new ClipboardHelper(getStackTrace(), clipboarButton);
-      buttonsLayout.add(clipboardHelper);
-      buttonsLayout.setAlignSelf(Alignment.START, clipboardHelper);
-      buttonsLayout.setFlexGrow(1.0, clipboardHelper);
+      clipboarButton.addClickListener(e -> UI.getCurrent().getPage()
+          .executeJs("navigator.clipboard.writeText($0)", getStackTrace()));
+      clipboarButton.getStyle().set("margin-right", "auto");
+      buttonsLayout.add(clipboarButton);
 
       // show details button
       Button button = createDetailsButtonLayout();