fix(Popup): ESC key press on trigger should propagate event if Popup is already closed

CHANGELOG.md

@@ -17,6 +17,9 @@ This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.htm
 
 ## [Unreleased]
 
+### Fixes
+- ESC key pressed on a trigger element should propagate event if `Popup` is closed @sophieH29 ([#1373](https://github.com/stardust-ui/react/pull/1373))
+
 ### Features
 - Add keyboard navigation and screen reader support for `Accordion` @silviuavram ([#1322](https://github.com/stardust-ui/react/pull/1322)) 
 - Add `expanded` prop to `Accordion` @silviuavram ([#1322](https://github.com/stardust-ui/react/pull/1322)) 

packages/react/src/components/Popup/Popup.tsx

@@ -208,11 +208,9 @@ export default class Popup extends AutoControlledComponent<PopupProps, PopupStat
   protected actionHandlers: AccessibilityActionHandlers = {
     closeAndFocusTrigger: e => {
       this.close(e, () => _.invoke(this.triggerFocusableDomElement, 'focus'))
-      e.stopPropagation()
     },
     close: e => {
       this.close(e)
-      e.stopPropagation()
     },
     toggle: e => {
       e.preventDefault()
@@ -552,6 +550,7 @@ export default class Popup extends AutoControlledComponent<PopupProps, PopupStat
     if (this.state.open) {
       this.trySetOpen(false, e)
       onClose && onClose()
+      e.stopPropagation()
     }
   }