removed e.stopPropagation() from onKeyDown (#2354)

* removed e.stopPropagation() from onKeyDown

* added check if propgation has already been stopped

* added test for isPropagationStopped

* removed stopPropagation completely, also removed the test

Co-authored-by: Daniel Lu <dlu@livefyre.com>
Co-authored-by: Robert Snow <rsnow@adobe.com>

Author: 3 people

packages/@react-aria/menu/src/useMenuTrigger.ts

@@ -56,12 +56,10 @@ export function useMenuTrigger(props: MenuTriggerAriaProps, state: MenuTriggerSt
         case 'Enter':
         case ' ':
           e.preventDefault();
-          e.stopPropagation();
           state.toggle('first');
           break;
         case 'ArrowUp':
           e.preventDefault();
-          e.stopPropagation();
           state.toggle('last');
           break;
       }

packages/@react-aria/menu/test/useMenuTrigger.test.js

@@ -114,7 +114,7 @@ describe('useMenuTrigger', function () {
     expect(setOpen).toHaveBeenCalledTimes(0);
     expect(setFocusStrategy).toHaveBeenCalledTimes(0);
 
-     // triggers event if defaultPrevented is not true and it matches one of the keys
+    // triggers event if defaultPrevented is not true and it matches one of the keys
     menuTriggerProps.onKeyDown({
       pointerType: 'not keyboard',
       defaultPrevented: false,