refactor: Remove deprecated props and related code

Author: aojunhao123

src/index.tsx

@@ -72,9 +72,6 @@ export interface TriggerProps {
   forceRender?: boolean;
   autoDestroy?: boolean;
 
-  /** @deprecated Please use `autoDestroy` instead */
-  destroyPopupOnHide?: boolean;
-
   // ==================== Mask =====================
   mask?: boolean;
   maskClosable?: boolean;
@@ -123,10 +120,6 @@ export interface TriggerProps {
   // ==================== Arrow ====================
   arrow?: boolean | ArrowTypeOuter;
 
-  // ================= Deprecated ==================
-  /** @deprecated Add `className` on `children`. Please add `className` directly instead. */
-  className?: string;
-
   // =================== Private ===================
   /**
    * @private Get trigger DOM node.
@@ -174,7 +167,6 @@ export function generateTrigger(
       getPopupContainer,
       forceRender,
       autoDestroy,
-      destroyPopupOnHide,
 
       // Popup
       popup,
@@ -205,16 +197,13 @@ export function generateTrigger(
       maskTransitionName,
       maskAnimation,
 
-      // Deprecated
-      className,
-
       // Private
       getTriggerDOMNode,
 
       ...restProps
     } = props;
 
-    const mergedAutoDestroy = autoDestroy || destroyPopupOnHide || false;
+    const mergedAutoDestroy = autoDestroy || false;
 
     // =========================== Mobile ===========================
     const [mobile, setMobile] = React.useState(false);
@@ -639,11 +628,6 @@ export function generateTrigger(
       };
     }
 
-    // ========================= ClassName ==========================
-    if (className) {
-      cloneProps.className = classNames(originChildProps.className, className);
-    }
-
     // =========================== Render ===========================
     const mergedChildrenProps = {
       ...originChildProps,

tests/basic.test.jsx

@@ -384,25 +384,6 @@ describe('Trigger.Basic', () => {
       trigger(container, '.target');
       expect(document.querySelector('.rc-trigger-popup-hidden')).toBeTruthy();
     });
-
-    it('set true will destroy tooltip on hide', () => {
-      const { container } = render(
-        <Trigger
-          action={['click']}
-          destroyPopupOnHide
-          popupAlign={placementAlignMap.topRight}
-          popup={<strong>trigger</strong>}
-        >
-          <div className="target">click</div>
-        </Trigger>,
-      );
-
-      trigger(container, '.target');
-      expect(document.querySelector('.rc-trigger-popup')).toBeTruthy();
-
-      trigger(container, '.target');
-      expect(document.querySelector('.rc-trigger-popup')).toBeFalsy();
-    });
   });
 
   describe('support autoDestroy', () => {
@@ -603,47 +584,6 @@ describe('Trigger.Basic', () => {
     expect(container.querySelector('div')).not.toHaveAttribute('class');
   });
 
-  it('support className', () => {
-    const { container } = render(
-      <Trigger
-        action={['click']}
-        popupAlign={placementAlignMap.left}
-        popup={<strong className="x-content">tooltip2</strong>}
-        className="className-in-trigger"
-      >
-        <div className="target">click</div>
-      </Trigger>,
-    );
-
-    expect(container.querySelector('div')).toHaveClass(
-      'target className-in-trigger',
-    );
-  });
-
-  it('support className in nested Trigger', () => {
-    const { container } = render(
-      <Trigger
-        action={['click']}
-        popupAlign={placementAlignMap.left}
-        popup={<strong className="x-content">tooltip2</strong>}
-        className="className-in-trigger-2"
-      >
-        <Trigger
-          action={['click']}
-          popupAlign={placementAlignMap.left}
-          popup={<strong className="x-content">tooltip2</strong>}
-          className="className-in-trigger-1"
-        >
-          <div className="target">click</div>
-        </Trigger>
-      </Trigger>,
-    );
-
-    expect(container.querySelector('div').className).toBe(
-      'target className-in-trigger-1 className-in-trigger-2',
-    );
-  });
-
   it('support function component', () => {
     const NoRef = React.forwardRef((props, ref) => {
       React.useImperativeHandle(ref, () => null);