Merge pull request #568 from IdelsTak/master

Introduce clearDragHover method for drag hover state resetting
torakiki · Dec 15, 2023 · d0d9ce0 · d0d9ce0
2 parents 472f185 + 00d2b65
commit d0d9ce0
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/...-components/src/main/java/org/pdfsam/ui/components/selection/multiple/SelectionTable.java b/...-components/src/main/java/org/pdfsam/ui/components/selection/multiple/SelectionTable.java
@@ -393,6 +393,7 @@ private Consumer<DragEvent> onDragEnteredConsumer() {
     }
 
     private void onDragExited(DragEvent e) {
+        clearDragHover();
         placeHolder.setDisable(true);
         e.consume();
     }
@@ -421,7 +422,7 @@ public void onLoadDocumentsRequest(PdfLoadRequest loadEvent) {
         getSortOrder().clear();
         getItems().addAll(dropIndex, toDrop);
         focus.map(getItems()::indexOf).ifPresent(getFocusModel()::focus);
-        hoverIndex.setValue(-1);
+        clearDragHover();
         this.sort();
 
         loadEvent.getDocuments().stream().findFirst().ifPresent(
@@ -558,4 +559,8 @@ private int calculateHoverIndex(DragEvent event) {
 
         return -1;
     }
+
+    private void clearDragHover() {
+        hoverIndex.setValue(-1);
+    }
 }