fix: fix double-click in read-only mode

Close #177
FlowingCode · Oct 9, 2024 · 9aaa8e2 · 9aaa8e2
1 parent aea4ebc
commit 9aaa8e2
Showing 1 changed file with 10 additions and 8 deletions.
diff --git a/src/main/java/com/flowingcode/vaadin/addons/twincolgrid/TwinColGrid.java b/src/main/java/com/flowingcode/vaadin/addons/twincolgrid/TwinColGrid.java
@@ -150,7 +150,7 @@ public enum Orientation {
   private boolean explicitHeaderRow = true;
 
   private String layoutId;
-  
+
   private static <T> ListDataProvider<T> emptyDataProvider() {
     return DataProvider.ofCollection(new LinkedHashSet<>());
   }
@@ -339,7 +339,7 @@ private String getLayoutId() {
       return this.layoutId = "twincol-" + UUID.randomUUID();
     });
   }
-  
+
   private HorizontalLayout createHorizontalContainer(boolean reverse) {
     buttonContainer = getVerticalButtonContainer();
     HorizontalLayout hl;
@@ -928,12 +928,14 @@ public void setMoveItemsByDoubleClick(boolean value) {
             side.moveItemsByDoubleClick =
                 side.grid.addItemDoubleClickListener(
                     ev -> {
-                      Set<T> item = Collections.singleton(ev.getItem());
-                      if (side == available) {
-                        updateSelection(item, Collections.emptySet(), true);
-                      }
-                      if (side == selection) {
-                        updateSelection(Collections.emptySet(), item, true);
+                      if (!isReadOnly()) {
+                        Set<T> item = Collections.singleton(ev.getItem());
+                        if (side == available) {
+                          updateSelection(item, Collections.emptySet(), true);
+                        }
+                        if (side == selection) {
+                          updateSelection(Collections.emptySet(), item, true);
+                        }
                       }
                     });
           }