refactor(accordion): ♻️ remove effect for defaultActiveId

adaptui · Sep 16, 2020 · 41597be · 41597be
1 parent 8b65c23
commit 41597be
Showing 1 changed file with 1 addition and 7 deletions.
diff --git a/src/accordion/AccordionState.ts b/src/accordion/AccordionState.ts
@@ -83,7 +83,7 @@ export function useAccordionState(
 
   const [state, dispatch] = React.useReducer(reducer, {
     items: [],
-    activeItems: [],
+    activeItems: defaultActiveId ? [defaultActiveId] : [],
     buttons: [],
     allowMultiple,
     loop,
@@ -127,12 +127,6 @@ export function useAccordionState(
     moveFocus(buttons[total - 1]);
   }, [buttons, total]);
 
-  React.useEffect(() => {
-    if (defaultActiveId) {
-      dispatch({ type: "addActiveItem", id: defaultActiveId });
-    }
-  }, [defaultActiveId]);
-
   return {
     ...idState,
     ...state,