Misc changes for the new tree- and -list api #3696

enonic · Sep 12, 2024 · 4b96802 · 4b96802
1 parent 3558375
commit 4b96802
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 14 deletions.
diff --git a/src/main/resources/assets/admin/common/js/ui/selector/list/FilterableListBoxWrapper.ts b/src/main/resources/assets/admin/common/js/ui/selector/list/FilterableListBoxWrapper.ts
@@ -151,7 +151,7 @@ export class FilterableListBoxWrapper<I>
                     const focusedItem: I = this.getFocusedItem();
 
                     if (focusedItem) {
-                        this.toggleSelection(focusedItem, this.isItemSelected(focusedItem));
+                        this.handleUserToggleAction(focusedItem);
                         return false;
                     }
 
@@ -243,11 +243,7 @@ export class FilterableListBoxWrapper<I>
 
         if (focusedItem) {
             if (this.selectionDelta.size === 0) {
-                if (this.isItemSelected(focusedItem)) {
-                    this.deselect(focusedItem);
-                } else {
-                    this.select(focusedItem);
-                }
+                this.handleUserToggleAction(focusedItem);
             }
 
             this.applySelection();

diff --git a/src/main/resources/assets/admin/common/js/ui/selector/list/SelectableListBoxWrapper.ts b/src/main/resources/assets/admin/common/js/ui/selector/list/SelectableListBoxWrapper.ts
@@ -296,14 +296,6 @@ export class SelectableListBoxWrapper<I>
         this.deselect(this.getCurrentlySelectedItems(), silent);
     }
 
-    toggleSelection(item: I, selected: boolean, silent?: boolean): void {
-        if (selected) {
-            this.deselect(item, silent);
-        } else {
-            this.select(item, silent);
-        }
-    }
-
     updateItem(item: I): void {
         this.listBox.replaceItems(item);
     }