fix(select): prevent modal form closing immediately when opened with Enter

Author: haryopba

core/src/components/select-modal/select-modal.tsx

@@ -94,12 +94,20 @@ export class SelectModal implements ComponentInterface {
               justify="start"
               labelPlacement="end"
               onClick={() => this.closeModal()}
+              onKeyDown={(ev) => {
+                if (ev.key === 'Enter') {
+                  /**
+                   * Selecting a radio option with keyboard navigation,
+                   * Enter key should dismiss the modal.
+                   */
+                  this.closeModal();
+                }
+              }}
               onKeyUp={(ev) => {
-                if (ev.key === 'Enter' || ev.key === ' ') {
+                if (ev.key === ' ') {
                   /**
                    * Selecting a radio option with keyboard navigation,
-                   * either through the Enter or Space keys, should
-                   * dismiss the modal.
+                   * Space key should dismiss the modal.
                    */
                   this.closeModal();
                 }

core/src/components/select-popover/select-popover.tsx

@@ -159,12 +159,20 @@ export class SelectPopover implements ComponentInterface {
               value={option.value}
               disabled={option.disabled}
               onClick={() => this.dismissParentPopover()}
+              onKeyDown={(ev) => {
+                if (ev.key === 'Enter') {
+                  /**
+                   * Selecting a radio option with keyboard navigation,
+                   * Enter key should dismiss the popover.
+                   */
+                  this.dismissParentPopover();
+                }
+              }}
               onKeyUp={(ev) => {
-                if (ev.key === 'Enter' || ev.key === ' ') {
+                if (ev.key === ' ') {
                   /**
                    * Selecting a radio option with keyboard navigation,
-                   * either through the Enter or Space keys, should
-                   * dismiss the popover.
+                   * Space key should dismiss the popover.
                    */
                   this.dismissParentPopover();
                 }