[react-interactions] Prevent duplicate onPress firing for keyboard Enter

Author: trueadm

packages/react-interactions/events/src/dom/Press.js

@@ -106,7 +106,10 @@ function isValidKey(e): boolean {
 
 function handlePreventDefault(preventDefault: ?boolean, e: any): void {
   const key = e.key;
-  if (preventDefault !== false && (key === ' ' || key === 'Enter')) {
+  if (
+    preventDefault !== false &&
+    (key === ' ' || key === 'Enter' || key === 'Spacebar')
+  ) {
     e.preventDefault();
   }
 }

packages/react-interactions/events/src/dom/PressLegacy.js

@@ -570,7 +570,7 @@ const pressResponderImpl = {
                 metaKey,
                 shiftKey,
               } = (nativeEvent: MouseEvent);
-              if (nativeEvent.key === ' ') {
+              if (isValidKeyboardEvent(nativeEvent)) {
                 nativeEvent.preventDefault();
               } else if (
                 props.preventDefault !== false &&

packages/react-interactions/events/src/dom/__tests__/PressLegacy-test.internal.js

@@ -914,8 +914,7 @@ describe.each(environmentTable)('Press responder', hasPointerEvents => {
       ReactDOM.render(<Component />, container);
 
       const target = createEventTarget(ref.current);
-      target.keydown({key: 'Enter'});
-      target.click({preventDefault});
+      target.keydown({key: 'Enter', preventDefault});
       target.keyup({key: 'Enter'});
       expect(preventDefault).toBeCalled();
       expect(onPress).toHaveBeenCalledWith(