[useButton] Stop setting type on non-native buttons (#4819)

Author: chuganzy

packages/react/src/internals/use-button/useButton.test.tsx

@@ -41,6 +41,19 @@ describe('useButton', () => {
         });
       });
 
+      it('does not set a type prop', async () => {
+        let buttonProps: Record<string, unknown> | undefined = undefined;
+
+        function Button() {
+          const { getButtonProps } = useButton({ native: false });
+          buttonProps = getButtonProps();
+          return <span {...buttonProps} />;
+        }
+
+        await render(<Button />);
+        expect(buttonProps).not.toHaveProperty('type');
+      });
+
       it.skipIf(isJSDOM)(
         'can be activated with Enter when the keyboard event originates inside a shadow root',
         async () => {

packages/react/src/internals/use-button/useButton.ts

@@ -98,11 +98,8 @@ export function useButton(parameters: UseButtonParameters = {}): UseButtonReturn
         ...otherExternalProps
       } = externalProps;
 
-      const type = isNativeButton ? 'button' : undefined;
-
       return mergeProps<'button'>(
         {
-          type,
           onClick(event: React.MouseEvent) {
             if (disabled) {
               event.preventDefault();
@@ -209,7 +206,7 @@ export function useButton(parameters: UseButtonParameters = {}): UseButtonReturn
             externalOnPointerDown?.(event);
           },
         },
-        !isNativeButton ? { role: 'button' } : undefined,
+        isNativeButton ? { type: 'button' } : { role: 'button' },
         focusableWhenDisabledProps,
         otherExternalProps,
       );