Fix selection loss in classifications view when moving

Bios-Marcel · Jul 15, 2024 · a315933 · a315933
1 parent f6a5ab7
commit a315933
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/src/main/java/link/biosmarcel/baka/view/ClassificationsView.java b/src/main/java/link/biosmarcel/baka/view/ClassificationsView.java
@@ -117,8 +117,9 @@ protected void updateItem(@Nullable ClassificationRuleFX item, boolean empty) {
             state.data.classificationRules.set(index, state.data.classificationRules.get(index - 1));
             state.data.classificationRules.set(index - 1, selectedClassification.rule);
 
-            // FIXME delta change
-            listView.getItems().setAll(convertRules(state.data.classificationRules));
+            final int oldViewIndex = listView.getItems().indexOf(selectedClassification);
+            listView.getItems().set(oldViewIndex, listView.getItems().get(oldViewIndex - 1));
+            listView.getItems().set(index - 1, selectedClassification);
             listView.getSelectionModel().select(selectedClassification);
         });
         final var downButton = new Button("↓");
@@ -136,8 +137,9 @@ protected void updateItem(@Nullable ClassificationRuleFX item, boolean empty) {
             state.data.classificationRules.set(index, state.data.classificationRules.get(index + 1));
             state.data.classificationRules.set(index + 1, selectedClassification.rule);
 
-            // FIXME delta change
-            listView.getItems().setAll(convertRules(state.data.classificationRules));
+            final int oldViewIndex = listView.getItems().indexOf(selectedClassification);
+            listView.getItems().set(oldViewIndex, listView.getItems().get(oldViewIndex + 1));
+            listView.getItems().set(index + 1, selectedClassification);
             listView.getSelectionModel().select(selectedClassification);
         });
         final var listViewButtons = new HBox(