refactor: move hover cleanup into Notification

Author: zombieJ

src/Notification.tsx

@@ -63,6 +63,7 @@ export interface NotificationProps {
   pauseOnHover?: boolean;
 
   // Function
+  onHover?: (hovering: boolean) => void;
   onClick?: React.MouseEventHandler<HTMLDivElement>;
   onMouseEnter?: React.MouseEventHandler<HTMLDivElement>;
   onMouseLeave?: React.MouseEventHandler<HTMLDivElement>;
@@ -99,6 +100,7 @@ const Notification = React.forwardRef<HTMLDivElement, NotificationProps>((props,
     pauseOnHover = true,
 
     // Function
+    onHover,
     onClick,
     onMouseEnter,
     onMouseLeave,
@@ -136,7 +138,18 @@ const Notification = React.forwardRef<HTMLDivElement, NotificationProps>((props,
   }, [forcedHovering, hovering, onPause, onResume, pauseOnHover]);
 
   // ========================= Hover ==========================
+  const hoveringRef = React.useRef(false);
+  const onInternalHover = useEvent((nextHovering: boolean) => {
+    if (nextHovering || hoveringRef.current) {
+      onHover?.(nextHovering);
+    }
+    hoveringRef.current = nextHovering;
+  });
+
+  React.useEffect(() => () => onInternalHover(false), [onInternalHover]);
+
   function onInternalMouseEnter(event: React.MouseEvent<HTMLDivElement>) {
+    onInternalHover(true);
     setHovering(true);
     if (pauseOnHover) {
       onPause();
@@ -145,6 +158,8 @@ const Notification = React.forwardRef<HTMLDivElement, NotificationProps>((props,
   }
 
   function onInternalMouseLeave(event: React.MouseEvent<HTMLDivElement>) {
+    // Normal exits are handled by the list, including movement through its gaps.
+    hoveringRef.current = false;
     setHovering(false);
     if (pauseOnHover && !forcedHovering) {
       onResume();

src/NotificationList/index.tsx

@@ -17,7 +17,7 @@ import Content from './Content';
 export type Placement = 'top' | 'topLeft' | 'topRight' | 'bottom' | 'bottomLeft' | 'bottomRight';
 export type { StackConfig } from '../hooks/useStack';
 
-export interface NotificationListConfig extends Omit<NotificationProps, 'prefixCls'> {
+export interface NotificationListConfig extends Omit<NotificationProps, 'prefixCls' | 'onHover'> {
   key: React.Key;
   placement?: Placement;
   times?: number;
@@ -139,15 +139,6 @@ const NotificationListItem: React.FC<NotificationListItemProps> = (props) => {
     [setNodeSize, strKey],
   );
   const ref = useComposeRef(nodeRef, setItemRef);
-  const hoveringRef = React.useRef(false);
-  const onInternalHover = useEvent((hovering: boolean) => {
-    if (hovering || hoveringRef.current) {
-      onHover(strKey, hovering);
-    }
-    hoveringRef.current = hovering;
-  });
-
-  React.useEffect(() => () => onInternalHover(false), [onInternalHover]);
 
   return (
     <Notification
@@ -164,15 +155,7 @@ const NotificationListItem: React.FC<NotificationListItemProps> = (props) => {
       }}
       hovering={stackEnabled && listHovering}
       pauseOnHover={config.pauseOnHover ?? pauseOnHover}
-      onMouseEnter={(event) => {
-        onInternalHover(true);
-        config.onMouseEnter?.(event);
-      }}
-      onMouseLeave={(event) => {
-        // The list handles normal exits, including movement through its gaps.
-        hoveringRef.current = false;
-        config.onMouseLeave?.(event);
-      }}
+      onHover={(hovering) => onHover(strKey, hovering)}
       onClose={() => {
         config.onClose?.();
         onNoticeClose?.(key);