feat: retire deprecated api

Author: aojunhao123

docs/demo/visible.md renamed to docs/demo/open.md

@@ -5,4 +5,4 @@ nav:
   path: /demo
 ---
 
-<code src="../../examples/visible.tsx"></code>
+<code src="../../examples/open.tsx"></code>

examples/visible.tsx renamed to examples/open.tsx

@@ -59,7 +59,7 @@ const addressOptions = [
 
 const Demo = () => {
   const [value, setValue] = useState<string[]>([]);
-  const [popupVisible, setPopupVisible] = useState(false);
+  const [open, setOpen] = useState(false);
 
   const getLabel = () => {
     return arrayTreeFilter(addressOptions, (o, level) => o.value === value[level])
@@ -69,10 +69,10 @@ const Demo = () => {
 
   return (
     <Cascader
-      popupVisible={popupVisible}
+      open={open}
       value={value}
       options={addressOptions}
-      onPopupVisibleChange={open => setPopupVisible(open)}
+      onPopupVisibleChange={open => setOpen(open)}
       onChange={value => setValue(value)}
     >
       <input value={getLabel()} readOnly />

src/Cascader.tsx

@@ -99,22 +99,18 @@ interface BaseCascaderProps<
   loadData?: (selectOptions: OptionType[]) => void;
 
   // Open
-  /** @deprecated Use `open` instead */
-  popupVisible?: boolean;
 
-  /** @deprecated Use `dropdownClassName` instead */
-  popupClassName?: string;
+  /** @deprecated Use `popupClassName` instead */
   dropdownClassName?: string;
+  popupClassName?: string;
   dropdownMenuColumnStyle?: React.CSSProperties;
 
-  /** @deprecated Use `placement` instead */
-  popupPlacement?: Placement;
   placement?: Placement;
   builtinPlacements?: BuildInPlacements;
 
-  /** @deprecated Use `onDropdownVisibleChange` instead */
-  onPopupVisibleChange?: (open: boolean) => void;
+  /** @deprecated Use `onPopupVisibleChange` instead */
   onDropdownVisibleChange?: (open: boolean) => void;
+  onPopupVisibleChange?: (open: boolean) => void;
 
   // Icon
   expandIcon?: React.ReactNode;
@@ -209,15 +205,13 @@ const Cascader = React.forwardRef<CascaderRef, InternalCascaderProps>((props, re
     loadData,
 
     // Open
-    popupVisible,
     open,
 
     popupClassName,
     dropdownClassName,
     dropdownMenuColumnStyle,
     dropdownStyle: customDropdownStyle,
 
-    popupPlacement,
     placement,
 
     onDropdownVisibleChange,
@@ -371,13 +365,8 @@ const Cascader = React.forwardRef<CascaderRef, InternalCascaderProps>((props, re
     onInternalSelect(valueCells);
   };
 
-  // ============================ Open ============================
-  const mergedOpen = open !== undefined ? open : popupVisible;
-
   const mergedDropdownClassName = dropdownClassName || popupClassName;
 
-  const mergedPlacement = placement || popupPlacement;
-
   const onInternalDropdownVisibleChange = (nextVisible: boolean) => {
     onDropdownVisibleChange?.(nextVisible);
     onPopupVisibleChange?.(nextVisible);
@@ -468,9 +457,9 @@ const Cascader = React.forwardRef<CascaderRef, InternalCascaderProps>((props, re
         OptionList={OptionList}
         emptyOptions={emptyOptions}
         // Open
-        open={mergedOpen}
+        open={open}
         dropdownClassName={mergedDropdownClassName}
-        placement={mergedPlacement}
+        placement={placement}
         onDropdownVisibleChange={onInternalDropdownVisibleChange}
         // Children
         getRawInputElement={() => children as React.ReactElement}

src/utils/warningPropsUtil.ts

@@ -2,20 +2,15 @@ import warning from 'rc-util/lib/warning';
 import type { DefaultOptionType, FieldNames, InternalCascaderProps } from '../Cascader';
 
 function warningProps(props: InternalCascaderProps) {
-  const { onPopupVisibleChange, popupVisible, popupClassName, popupPlacement } = props;
+  const { onDropdownVisibleChange, dropdownClassName } = props;
 
   warning(
-    !onPopupVisibleChange,
-    '`onPopupVisibleChange` is deprecated. Please use `onDropdownVisibleChange` instead.',
+    !onDropdownVisibleChange,
+    '`onDropdownVisibleChange` is deprecated. Please use `onPopupVisibleChange` instead.',
   );
-  warning(popupVisible === undefined, '`popupVisible` is deprecated. Please use `open` instead.');
   warning(
-    popupClassName === undefined,
-    '`popupClassName` is deprecated. Please use `dropdownClassName` instead.',
-  );
-  warning(
-    popupPlacement === undefined,
-    '`popupPlacement` is deprecated. Please use `placement` instead.',
+    dropdownClassName === undefined,
+    '`dropdownClassName` is deprecated. Please use `popupClassName` instead.',
   );
 }
 

tests/index.spec.tsx

@@ -514,30 +514,24 @@ describe('Cascader.Basic', () => {
     expect(wrapper.isOpen()).toBeTruthy();
   });
 
-  it('warning popupVisible & onPopupVisibleChange & popupClassName', () => {
+  it('onDropdownVisibleChange & dropdownClassName', () => {
     resetWarned();
     const errorSpy = jest.spyOn(console, 'error').mockImplementation(() => {});
-    const onPopupVisibleChange = jest.fn();
+    const onDropdownVisibleChange = jest.fn();
     const wrapper = mount(
       <Cascader
-        popupVisible
-        onPopupVisibleChange={onPopupVisibleChange}
-        popupClassName="legacy-cls"
-        popupPlacement="topRight"
+        open
+        onDropdownVisibleChange={onDropdownVisibleChange}
+        dropdownClassName="legacy-cls"
+        placement="topRight"
       />,
     );
 
     expect(errorSpy).toHaveBeenCalledWith(
-      'Warning: `onPopupVisibleChange` is deprecated. Please use `onDropdownVisibleChange` instead.',
-    );
-    expect(errorSpy).toHaveBeenCalledWith(
-      'Warning: `popupVisible` is deprecated. Please use `open` instead.',
+      'Warning: `onDropdownVisibleChange` is deprecated. Please use `onPopupVisibleChange` instead.',
     );
     expect(errorSpy).toHaveBeenCalledWith(
-      'Warning: `popupClassName` is deprecated. Please use `dropdownClassName` instead.',
-    );
-    expect(errorSpy).toHaveBeenCalledWith(
-      'Warning: `popupPlacement` is deprecated. Please use `placement` instead.',
+      'Warning: `dropdownClassName` is deprecated. Please use `popupClassName` instead.',
     );
 
     expect(wrapper.exists('.legacy-cls')).toBeTruthy();
@@ -610,7 +604,7 @@ describe('Cascader.Basic', () => {
       },
     ];
     const wrapper = mount(
-      <Cascader options={options} popupVisible>
+      <Cascader options={options} open>
         <input readOnly />
       </Cascader>,
     );
@@ -622,7 +616,7 @@ describe('Cascader.Basic', () => {
     const wrapper = mount(
       <Cascader
         options={addressOptions}
-        popupVisible
+        open
         dropdownRender={menus => (
           <div className="custom-dropdown">
             {menus}