fix: support removing more than one pane at a time (#320)

johnwalley · Jun 6, 2022 · 8b3b444 · 8b3b444
1 parent 76954bf
commit 8b3b444
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/allotment.tsx b/src/allotment.tsx
@@ -294,13 +294,13 @@ const Allotment = forwardRef<AllotmentHandle, AllotmentProps>(
         const update = keys.filter((key) => previousKeys.current.includes(key));
         const exit = previousKeys.current.map((key) => !keys.includes(key));
 
-        exit.forEach((flag, index) => {
-          if (flag) {
+        for (let index = exit.length - 1; index >= 0; index--) {
+          if (exit[index]) {
             splitViewRef.current?.removeView(index);
             panes.splice(index, 1);
             views.current.splice(index, 1);
           }
-        });
+        }
 
         for (const enterKey of enter) {
           const props = splitViewPropsRef.current.get(enterKey);