Merge pull request #7092 from alibaba/fix/weex-appear-cb

fix: fix appear event handler callback refs in WeexAppear component

Author: Psiphonc

.changeset/fast-falcons-serve.md

@@ -0,0 +1,5 @@
+---
+'@ice/appear': patch
+---
+
+fix appear event handler callback refs in WeexAppear component

packages/appear/src/weex/index.tsx

@@ -12,21 +12,28 @@ const WeexAppear = forwardRef<any, AppearProps>((props, ref) => {
     // Use copy of childrenRef to avoid ref value changed in cleanup phase.
     const nodeRef = typeof childrenRef === 'object' ? childrenRef.current : null;
 
+    const appearHandler = (e: CustomEvent) => {
+      onAppear?.(e);
+    };
     // Return early if onAppear callback not specified.
-    onAppear && nodeRef?.addEventListener('appear', (e: CustomEvent) => onAppear(e));
+    onAppear && nodeRef?.addEventListener('appear', appearHandler);
 
     return () => {
-      onAppear && nodeRef?.removeEventListener('appear', (e: CustomEvent) => onAppear(e));
+      onAppear && nodeRef?.removeEventListener('appear', appearHandler);
     };
   }, [childrenRef, onAppear]);
 
   useEffect(() => {
     const nodeRef = typeof childrenRef === 'object' ? childrenRef.current : null;
 
-    onDisappear && nodeRef?.addEventListener('disappear', (e: CustomEvent) => onDisappear(e));
+    const disappearHandler = (e: CustomEvent) => {
+      onDisappear?.(e);
+    };
+
+    onDisappear && nodeRef?.addEventListener('disappear', disappearHandler);
 
     return () => {
-      onDisappear && nodeRef?.removeEventListener('disappear', (e: CustomEvent) => onDisappear(e));
+      onDisappear && nodeRef?.removeEventListener('disappear', disappearHandler);
     };
   }, [childrenRef, onDisappear]);