Merge 19149b0 into b5c808a

maykinmedia · Dec 8, 2017 · d2bffd2 · d2bffd2
2 parents b5c808a + 19149b0
commit d2bffd2
Show file tree

Hide file tree

Showing 4 changed files with 132 additions and 8 deletions.
diff --git a/dist/dual-listbox.js b/dist/dual-listbox.js
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "dual-listbox",
-  "version": "1.0.9",
+  "version": "1.1.0",
   "description": "Dual listbox for multi-select elements",
   "main": "dist/dual-listbox.js",
   "directories": {

diff --git a/src/dual-listbox.js b/src/dual-listbox.js
@@ -159,9 +159,8 @@ class DualListbox {
     _actionAllSelected(event) {
         event.preventDefault();
 
-        while (this.available.length > 0) {
-            this.addSelected(this.available[0]);
-        }
+        let selected = this.available.filter((item) => item.style.display !== "none");
+        selected.forEach((item) => this.addSelected(item));
     }
 
     /**
@@ -193,9 +192,8 @@ class DualListbox {
     _actionAllDeselected(event) {
         event.preventDefault();
 
-        while (this.selected.length > 0) {
-            this.removeSelected(this.selected[0]);
-        }
+        let deselected = this.selected.filter((item) => item.style.display !== "none");
+        deselected.forEach((item) => this.removeSelected(item));
     }
 
     /**