Răsfoiți Sursa

fix: the onOpenChange event for the modal component does not work as expected (#4425)

Vben 6 luni în urmă
părinte
comite
834cb4c470

+ 1 - 0
internal/vite-config/src/config/application.ts

@@ -112,6 +112,7 @@ function createCssOptions(injectGlobalScss = true) {
               }
               return content;
             },
+            api: 'modern-compiler',
           },
         }
       : {},

+ 4 - 4
packages/@core/ui-kit/popup-ui/src/drawer/use-drawer.ts

@@ -54,10 +54,10 @@ export function useVbenDrawer<
     ...options,
   } as DrawerApiOptions;
 
-  // mergedOptions.onOpenChange = (isOpen: boolean) => {
-  //   options.onOpenChange?.(isOpen);
-  //   injectData.options?.onOpenChange?.(isOpen);
-  // };
+  mergedOptions.onOpenChange = (isOpen: boolean) => {
+    options.onOpenChange?.(isOpen);
+    injectData.options?.onOpenChange?.(isOpen);
+  };
   const api = new DrawerApi(mergedOptions);
 
   const extendedApi: ExtendedDrawerApi = api as never;

+ 4 - 4
packages/@core/ui-kit/popup-ui/src/modal/use-modal.ts

@@ -58,10 +58,10 @@ export function useVbenModal<TParentModalProps extends ModalProps = ModalProps>(
     ...options,
   } as ModalApiOptions;
 
-  // mergedOptions.onOpenChange = (isOpen: boolean) => {
-  //   options.onOpenChange?.(isOpen);
-  //   injectData.options?.onOpenChange?.(isOpen);
-  // };
+  mergedOptions.onOpenChange = (isOpen: boolean) => {
+    options.onOpenChange?.(isOpen);
+    injectData.options?.onOpenChange?.(isOpen);
+  };
   const api = new ModalApi(mergedOptions);
 
   const extendedApi: ExtendedModalApi = api as never;