fix: notification should hide close icon when closeIcon={null} (#42791)

components/notification/__tests__/index.test.tsx

@@ -272,6 +272,30 @@ describe('notification', () => {
     expect(document.querySelectorAll('.test-customize-icon').length).toBe(1);
   });
 
+  it('support closeIcon to be null', () => {
+    act(() => {
+      notification.open({
+        message: 'Notification Title',
+        duration: 0,
+        closeIcon: null,
+      });
+    });
+
+    expect(document.querySelectorAll('.test-customize-icon').length).toBe(0);
+  });
+
+  it('support closeIcon to be false', () => {
+    act(() => {
+      notification.open({
+        message: 'Notification Title',
+        duration: 0,
+        closeIcon: false,
+      });
+    });
+
+    expect(document.querySelectorAll('.test-customize-icon').length).toBe(0);
+  });
+
   it('support config closeIcon', () => {
     notification.config({
       closeIcon: <span className="test-customize-icon" />,

components/notification/index.tsx

@@ -252,11 +252,14 @@ function getRCNoticeProps(args: ArgsProps, prefixCls: string, iconPrefixCls?: st
     });
   }
 
-  const closeIconToRender = (
+  const closeIconToRender =
-    <span className={`${prefixCls}-close-x`}>
+    typeof closeIcon === 'undefined' ? (
-      {closeIcon || <CloseOutlined className={`${prefixCls}-close-icon`} />}
+      <span className={`${prefixCls}-close-x`}>
-    </span>
+        <CloseOutlined className={`${prefixCls}-close-icon`} />
-  );
+      </span>
+    ) : (
+      closeIcon
+    );
 
   const autoMarginTag =
     !description && iconNode ? (