@appcorp/fusion-storybook 0.1.101 → 0.2.0

package/base-modules/admission/page.js

@@ -52,7 +52,7 @@ const createComponentInstances = () => ({
     moreActions: _jsx(AdmissionMoreActions, {}),
     view: _jsx(AdmissionView, {}),
 });
-const createAdmissionConfig = ({ cancelLabel, dispatch, drawer, drawerTitle, labelActions, labelClass, labelEnabled, labelFirstName, labelId, labelLastName, labelRegistrationCode, labelStatus, saveLabel, searchPlaceholder, tableDescription, tableTitle, }) => {
+const createAdmissionConfig = ({ cancelLabel, dispatch, drawerTitle, labelActions, labelClass, labelEnabled, labelFirstName, labelId, labelLastName, labelRegistrationCode, labelStatus, saveLabel, searchPlaceholder, tableDescription, tableTitle, }) => {
     const components = createComponentInstances();
     return {
         moduleName: "admission",
@@ -76,7 +76,7 @@ const createAdmissionConfig = ({ cancelLabel, dispatch, drawer, drawerTitle, lab
         tableDescription,
         tableTitle,
         viewContent: components.view,
-        size: drawer === ADMISSION_DRAWER.FORM_DRAWER ? "full" : "small",
+        size: "small",
         onClearFilters: () => {
             dispatch({ type: ADMISSION_ACTION_TYPES.RESET_FORM });
         },
@@ -88,10 +88,12 @@ const createAdmissionConfig = ({ cancelLabel, dispatch, drawer, drawerTitle, lab
 const GenericAdmissionPage = createGenericModulePage();
 const AdmissionPageInner = (props) => {
     const context = useAdmissionModule();
-    // Memoize config creation - destructure props to avoid object reference changes
+    // Memoize config creation - INTENTIONALLY EXCLUDE drawer state to prevent infinite loops
+    // The drawer state changing (from clicking buttons) would recreate this config,
+    // causing unnecessary re-renders and state updates. Since config should be stable
+    // across drawer state changes, we exclude it from dependencies.
     const admissionConfig = useMemo(() => createAdmissionConfig({
         dispatch: context.dispatch,
-        drawer: context.state.drawer,
         cancelLabel: props.cancelLabel,
         drawerTitle: props.drawerTitle,
         labelActions: props.labelActions,
@@ -108,7 +110,6 @@ const AdmissionPageInner = (props) => {
         tableTitle: props.tableTitle,
     }), [
         context.dispatch,
-        context.state.drawer,
         props.cancelLabel,
         props.drawerTitle,
         props.labelActions,
@@ -124,6 +125,11 @@ const AdmissionPageInner = (props) => {
         props.tableDescription,
         props.tableTitle,
     ]);
+    // Dynamically apply drawer size to config for proper drawer sizing
+    // This allows size to change without recreating the entire config
+    const admissionConfigWithSize = useMemo(() => (Object.assign(Object.assign({}, admissionConfig), { size: context.state.drawer === ADMISSION_DRAWER.FORM_DRAWER
+            ? "full"
+            : "small" })), [admissionConfig, context.state.drawer]);
     const hasPermission = resolveRbacPermissions({
         userRole: props.userRole,
         moduleName: "Admission",
@@ -131,7 +137,7 @@ const AdmissionPageInner = (props) => {
     if (!hasPermission) {
         return _jsx(RbacNoAccess, { moduleName: "Admission" });
     }
-    return (_jsx("div", { className: "p-4", children: _jsx(GenericAdmissionPage, { overrideConfig: admissionConfig, context: context, tableBodyCols: tableBodyCols }) }));
+    return (_jsx("div", { className: "p-4", children: _jsx(GenericAdmissionPage, { overrideConfig: admissionConfigWithSize, context: context, tableBodyCols: tableBodyCols }) }));
 };
 // ============================================================================
 // PAGE EXPORTS

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@appcorp/fusion-storybook",
-  "version": "0.1.101",
+  "version": "0.2.0",
   "scripts": {
     "build-storybook": "storybook build",
     "build:next": "next build",