Remove reduce in Navigation state change

Author: chenglou

src/components/Navigation.mjs

@@ -527,21 +527,20 @@ function Navigation(Props) {
         }), []);
   var fixedNav = fixed ? "fixed top-0" : "relative";
   var onStateChange = function (id, state) {
-    Curry._1(setCollapsibles, (function (prev) {
-            return prev.reduce((function (acc, next) {
-                          if (next.title === id) {
-                            acc.push({
-                                  title: next.title,
-                                  children: next.children,
-                                  isActiveRoute: next.isActiveRoute,
-                                  href: next.href,
-                                  state: state
-                                });
-                          }
-                          return acc;
-                        }), []);
-          }));
-    
+    return Curry._1(setCollapsibles, (function (prev) {
+                  return Belt_Array.keepMap(prev, (function (next) {
+                                if (next.title === id) {
+                                  return {
+                                          title: next.title,
+                                          children: next.children,
+                                          isActiveRoute: next.isActiveRoute,
+                                          href: next.href,
+                                          state: state
+                                        };
+                                }
+                                
+                              }));
+                }));
   };
   var collapsibleElements = collapsibles.map(function (coll) {
         return React.createElement(Navigation$CollapsibleLink, {

src/components/Navigation.res

@@ -493,26 +493,14 @@ let make = (~fixed=true, ~overlayState: (bool, (bool => bool) => unit)) => {
 
   let onStateChange = (~id, state) => {
     setCollapsibles(prev => {
-      Js.Array2.reduce(
-        prev,
-        (acc, next) => {
-          if next.title === id {
-            acc
-            ->Js.Array2.push({
-              ...next,
-              state: state,
-            })
-            ->ignore
-          } else {
-            ()
-          }
-
-          acc
-        },
-        [],
-      )
+      Belt.Array.keepMap(prev, next=> {
+        if next.title === id {
+          Some({...next, state: state})
+        } else {
+          None
+        }
+      })
     })
-    ()
   }
 
   let collapsibleElements = Js.Array2.map(collapsibles, coll => {