fix: Button default event handler (#43666)

* test: test driven

* fix: types

* chore: post script

components/button/__tests__/index.test.tsx

@@ -351,7 +351,38 @@ describe('Button', () => {
     const button = <Button onClick={onBtnClick} />;
     const anchor = <Button href="https://ant.design" onClick={onAnchorClick} />;
 
+    const defaultBtn = (
+      <Button
+        onClick={(e) => {
+          expect(e).toBeTruthy();
+        }}
+      />
+    );
+
+    const defaultABtn = (
+      <Button
+        href="https://ant.design"
+        onClick={(e) => {
+          expect(e).toBeTruthy();
+        }}
+      />
+    );
+
+    const btnRef = React.createRef<HTMLButtonElement>();
+    const refBtn = <Button ref={btnRef} />;
+
+    const anchorRef = React.createRef<HTMLAnchorElement>();
+    const refAnchor = <Button ref={anchorRef} />;
+
+    const htmlRef = React.createRef<HTMLElement>();
+    const refHtml = <Button ref={htmlRef} />;
+
     expect(button).toBeTruthy();
     expect(anchor).toBeTruthy();
+    expect(defaultBtn).toBeTruthy();
+    expect(defaultABtn).toBeTruthy();
+    expect(refBtn).toBeTruthy();
+    expect(refAnchor).toBeTruthy();
+    expect(refHtml).toBeTruthy();
   });
 });

components/button/button.tsx

@@ -55,22 +55,15 @@ export interface BaseButtonProps {
   styles?: { icon: React.CSSProperties };
 }
 
-export type AnchorButtonProps = {
+type MergedHTMLAttributes = Omit<
-  href: string;
+  React.HTMLAttributes<HTMLElement> & React.AnchorHTMLAttributes<HTMLElement>,
-  target?: React.HTMLAttributeAnchorTarget;
+  'type'
-  onClick?: React.MouseEventHandler<HTMLAnchorElement>;
+>;
-} & BaseButtonProps &
-  Omit<React.AnchorHTMLAttributes<HTMLAnchorElement | HTMLButtonElement>, 'type' | 'onClick'>;
-
-export type NativeButtonProps = {
-  htmlType?: ButtonHTMLType;
-  onClick?: React.MouseEventHandler<HTMLButtonElement>;
-} & BaseButtonProps &
-  Omit<React.ButtonHTMLAttributes<HTMLButtonElement>, 'type' | 'onClick'>;
-
-export type ButtonProps = AnchorButtonProps | NativeButtonProps;
 
-type InternalButtonProps = Partial<AnchorButtonProps & NativeButtonProps>;
+export interface ButtonProps extends BaseButtonProps, MergedHTMLAttributes {
+  href?: string;
+  htmlType?: ButtonHTMLType;
+}
 
 type CompoundedComponent = React.ForwardRefExoticComponent<
   ButtonProps & React.RefAttributes<HTMLElement>
@@ -125,7 +118,7 @@ const InternalButton: React.ForwardRefRenderFunction<
     classNames: customClassNames,
     style: customStyle = {},
     ...rest
-  } = props as InternalButtonProps;
+  } = props;
 
   const { getPrefixCls, autoInsertSpaceInButton, direction, button } = useContext(ConfigContext);
   const prefixCls = getPrefixCls('btn', customizePrefixCls);
@@ -218,7 +211,7 @@ const InternalButton: React.ForwardRefRenderFunction<
 
   const iconType = innerLoading ? 'loading' : icon;
 
-  const linkButtonRestProps = omit(rest as InternalButtonProps & { navigate: any }, ['navigate']);
+  const linkButtonRestProps = omit(rest as ButtonProps & { navigate: any }, ['navigate']);
 
   const classes = classNames(
     prefixCls,
@@ -280,7 +273,7 @@ const InternalButton: React.ForwardRefRenderFunction<
 
   let buttonNode = (
     <button
-      {...(rest as NativeButtonProps)}
+      {...rest}
       type={htmlType}
       className={classes}
       style={fullStyle}

scripts/post-script.js

@@ -39,6 +39,7 @@ const DEPRECIATED_VERSION = {
   ],
   '5.6.2': ['https://github.com/ant-design/ant-design/issues/43113'],
   '5.6.3': ['https://github.com/ant-design/ant-design/issues/43190'],
+  '5.7.1': ['https://github.com/ant-design/ant-design/issues/43654'],
 };
 
 function matchDeprecated(version) {