From 55a8cf331ee33d6f51bd6cfefb2fab1c62e2bc75 Mon Sep 17 00:00:00 2001
From: ShaneK <shane@shanessite.net>
Date: Thu, 23 Apr 2026 09:12:18 -0700
Subject: [PATCH 1/2] fix(select): commit value and dismiss on Enter for
 popover and modal interfaces

---
 .../components/radio-group/radio-group.tsx    |  13 ++
 .../radio-group/test/basic/radio-group.e2e.ts |  26 ++++
 .../components/radio-group/test/fixtures.ts   |   4 +-
 .../components/select-modal/select-modal.tsx  |  22 +++-
 .../test/basic/select-modal.e2e.ts            |  29 +++++
 .../components/select-modal/test/fixtures.ts  |   5 +
 .../select-popover/select-popover.tsx         |  23 +++-
 .../test/basic/select-popover.e2e.ts          |  29 +++++
 .../select-popover/test/fixtures.ts           |   5 +
 .../select/test/basic/select.e2e.ts           | 123 ++++++++++++++++++
 10 files changed, 267 insertions(+), 12 deletions(-)

diff --git a/core/src/components/radio-group/radio-group.tsx b/core/src/components/radio-group/radio-group.tsx
index 88ff48e2c4c..37e0f0da0c9 100644
--- a/core/src/components/radio-group/radio-group.tsx
+++ b/core/src/components/radio-group/radio-group.tsx
@@ -290,6 +290,19 @@ export class RadioGroup implements ComponentInterface {
         // to the bottom of the screen
         ev.preventDefault();
       }
+
+      // Inside a select interface, Enter commits the focused radio
+      // value (matching native <select>). The !ev.repeat guard stops
+      // a held Enter on the triggering ion-select from re-committing
+      // once focus lands in the opened popover/modal.
+      if (ev.key === 'Enter' && inSelectInterface && !ev.repeat) {
+        const previousValue = this.value;
+        this.value = current.value;
+        if (previousValue !== this.value) {
+          this.emitValueChange(ev);
+        }
+        ev.preventDefault();
+      }
     }
   }
 
diff --git a/core/src/components/radio-group/test/basic/radio-group.e2e.ts b/core/src/components/radio-group/test/basic/radio-group.e2e.ts
index 96b7b5d9556..cea527e62d6 100644
--- a/core/src/components/radio-group/test/basic/radio-group.e2e.ts
+++ b/core/src/components/radio-group/test/basic/radio-group.e2e.ts
@@ -46,6 +46,32 @@ configs({ modes: ['ios'], directions: ['ltr'] }).forEach(({ title, config }) =>
       await radioFixture.expectChecked(false);
     });
 
+    test('Enter outside a select interface should not change the value', async ({ page }, testInfo) => {
+      testInfo.annotations.push({
+        type: 'issue',
+        description: 'https://github.com/ionic-team/ionic-framework/issues/30561',
+      });
+
+      await page.setContent(
+        `
+        <ion-radio-group value="one">
+          <ion-item>
+            <ion-radio id="one" value="one">One</ion-radio>
+          </ion-item>
+          <ion-item>
+            <ion-radio id="two" value="two">Two</ion-radio>
+          </ion-item>
+        </ion-radio-group>
+      `,
+        config
+      );
+
+      await radioFixture.checkRadio('keyboard', 'ion-radio#two', 'Enter');
+
+      const group = page.locator('ion-radio-group');
+      await expect(group).toHaveJSProperty('value', 'one');
+    });
+
     test('click should not deselect without allowEmptySelection', async ({ page }) => {
       await page.setContent(
         `
diff --git a/core/src/components/radio-group/test/fixtures.ts b/core/src/components/radio-group/test/fixtures.ts
index 21722126d7f..c8d5935a03c 100644
--- a/core/src/components/radio-group/test/fixtures.ts
+++ b/core/src/components/radio-group/test/fixtures.ts
@@ -11,13 +11,13 @@ export class RadioFixture {
     this.page = page;
   }
 
-  async checkRadio(method: 'keyboard' | 'mouse', selector = 'ion-radio') {
+  async checkRadio(method: 'keyboard' | 'mouse', selector = 'ion-radio', key: 'Space' | 'Enter' = 'Space') {
     const { page } = this;
     const radio = (this.radio = page.locator(selector));
 
     if (method === 'keyboard') {
       await radio.focus();
-      await page.keyboard.press('Space');
+      await page.keyboard.press(key);
     } else {
       await radio.click();
     }
diff --git a/core/src/components/select-modal/select-modal.tsx b/core/src/components/select-modal/select-modal.tsx
index d83d7fa76b0..74358033caf 100644
--- a/core/src/components/select-modal/select-modal.tsx
+++ b/core/src/components/select-modal/select-modal.tsx
@@ -21,6 +21,11 @@ import type { SelectModalOption } from './select-modal-interface';
 export class SelectModal implements ComponentInterface {
   @Element() el!: HTMLIonSelectModalElement;
 
+  // Tracks the option that received Enter-keydown so keyup only
+  // dismisses when the press started on the same option. Prevents
+  // Enter on the triggering ion-select from auto-dismissing.
+  private pendingEnterTarget: HTMLElement | null = null;
+
   @Prop() header?: string;
 
   /**
@@ -99,14 +104,21 @@ export class SelectModal implements ComponentInterface {
               justify="start"
               labelPlacement="end"
               onClick={() => this.closeModal()}
+              onKeyDown={(ev) => {
+                if (ev.key === 'Enter' && !ev.repeat) {
+                  this.pendingEnterTarget = ev.currentTarget as HTMLElement;
+                }
+              }}
               onKeyUp={(ev) => {
                 if (ev.key === ' ') {
-                  /**
-                   * Selecting a radio option with keyboard navigation,
-                   * either through the Enter or Space keys, should
-                   * dismiss the modal.
-                   */
+                  // Space selects and dismisses in one press.
                   this.closeModal();
+                } else if (ev.key === 'Enter') {
+                  const shouldClose = this.pendingEnterTarget === ev.currentTarget;
+                  this.pendingEnterTarget = null;
+                  if (shouldClose) {
+                    this.closeModal();
+                  }
                 }
               }}
             >
diff --git a/core/src/components/select-modal/test/basic/select-modal.e2e.ts b/core/src/components/select-modal/test/basic/select-modal.e2e.ts
index 126082712c3..11eb4270af9 100644
--- a/core/src/components/select-modal/test/basic/select-modal.e2e.ts
+++ b/core/src/components/select-modal/test/basic/select-modal.e2e.ts
@@ -64,6 +64,35 @@ configs({ modes: ['md'], directions: ['ltr'] }).forEach(({ title, config }) => {
         await expect(selectModalPage.modal).not.toBeVisible();
       });
 
+      // eslint-disable-next-line @typescript-eslint/no-unused-vars
+      test('pressing Enter on an unselected option should dismiss the modal', async ({ page: _page }, testInfo) => {
+        testInfo.annotations.push({
+          type: 'issue',
+          description: 'https://github.com/ionic-team/ionic-framework/issues/30561',
+        });
+
+        await selectModalPage.setup(config, options, false);
+
+        await selectModalPage.pressEnterOnOption('apple');
+        await selectModalPage.ionModalDidDismiss.next();
+        await expect(selectModalPage.modal).not.toBeVisible();
+      });
+
+      test('pressing Enter on a selected option should dismiss the modal', async ({ browserName }, testInfo) => {
+        testInfo.annotations.push({
+          type: 'issue',
+          description: 'https://github.com/ionic-team/ionic-framework/issues/30561',
+        });
+
+        test.skip(browserName === 'firefox', 'Same behavior as ROU-5437');
+
+        await selectModalPage.setup(config, checkedOptions, false);
+
+        await selectModalPage.pressEnterOnOption('apple');
+        await selectModalPage.ionModalDidDismiss.next();
+        await expect(selectModalPage.modal).not.toBeVisible();
+      });
+
       test('clicking the close button should dismiss the modal', async () => {
         await selectModalPage.setup(config, options, false);
 
diff --git a/core/src/components/select-modal/test/fixtures.ts b/core/src/components/select-modal/test/fixtures.ts
index 2058848aa84..e1df642c78b 100644
--- a/core/src/components/select-modal/test/fixtures.ts
+++ b/core/src/components/select-modal/test/fixtures.ts
@@ -63,6 +63,11 @@ export class SelectModalPage {
     await option.press('Space');
   }
 
+  async pressEnterOnOption(value: string) {
+    const option = this.getOption(value);
+    await option.press('Enter');
+  }
+
   private getOption(value: string) {
     const { multiple, selectModal } = this;
     const selector = multiple ? 'ion-checkbox' : 'ion-radio';
diff --git a/core/src/components/select-popover/select-popover.tsx b/core/src/components/select-popover/select-popover.tsx
index 00a8ccf6f2e..9e9b3fb13b5 100644
--- a/core/src/components/select-popover/select-popover.tsx
+++ b/core/src/components/select-popover/select-popover.tsx
@@ -22,6 +22,12 @@ import type { SelectPopoverOption } from './select-popover-interface';
 })
 export class SelectPopover implements ComponentInterface {
   @Element() el!: HTMLIonSelectPopoverElement;
+
+  // Tracks the option that received Enter-keydown so keyup only
+  // dismisses when the press started on the same option. Prevents
+  // Enter on the triggering ion-select from auto-dismissing.
+  private pendingEnterTarget: HTMLElement | null = null;
+
   /**
    * The header text of the popover
    */
@@ -159,14 +165,21 @@ export class SelectPopover implements ComponentInterface {
               value={option.value}
               disabled={option.disabled}
               onClick={() => this.dismissParentPopover()}
+              onKeyDown={(ev) => {
+                if (ev.key === 'Enter' && !ev.repeat) {
+                  this.pendingEnterTarget = ev.currentTarget as HTMLElement;
+                }
+              }}
               onKeyUp={(ev) => {
                 if (ev.key === ' ') {
-                  /**
-                   * Selecting a radio option with keyboard navigation,
-                   * either through the Enter or Space keys, should
-                   * dismiss the popover.
-                   */
+                  // Space selects and dismisses in one press.
                   this.dismissParentPopover();
+                } else if (ev.key === 'Enter') {
+                  const shouldDismiss = this.pendingEnterTarget === ev.currentTarget;
+                  this.pendingEnterTarget = null;
+                  if (shouldDismiss) {
+                    this.dismissParentPopover();
+                  }
                 }
               }}
             >
diff --git a/core/src/components/select-popover/test/basic/select-popover.e2e.ts b/core/src/components/select-popover/test/basic/select-popover.e2e.ts
index 14cc0c0c146..b2cb17bdaf8 100644
--- a/core/src/components/select-popover/test/basic/select-popover.e2e.ts
+++ b/core/src/components/select-popover/test/basic/select-popover.e2e.ts
@@ -63,6 +63,35 @@ configs({ modes: ['md'], directions: ['ltr'] }).forEach(({ title, config }) => {
         await selectPopoverPage.ionPopoverDidDismiss.next();
         await expect(selectPopoverPage.popover).not.toBeVisible();
       });
+
+      // eslint-disable-next-line @typescript-eslint/no-unused-vars
+      test('pressing Enter on an unselected option should dismiss the popover', async ({ page: _page }, testInfo) => {
+        testInfo.annotations.push({
+          type: 'issue',
+          description: 'https://github.com/ionic-team/ionic-framework/issues/30561',
+        });
+
+        await selectPopoverPage.setup(config, options, false);
+
+        await selectPopoverPage.pressEnterOnOption('apple');
+        await selectPopoverPage.ionPopoverDidDismiss.next();
+        await expect(selectPopoverPage.popover).not.toBeVisible();
+      });
+
+      test('pressing Enter on a selected option should dismiss the popover', async ({ browserName }, testInfo) => {
+        testInfo.annotations.push({
+          type: 'issue',
+          description: 'https://github.com/ionic-team/ionic-framework/issues/30561',
+        });
+
+        test.skip(browserName === 'firefox', 'Same behavior as ROU-5437');
+
+        await selectPopoverPage.setup(config, checkedOptions, false);
+
+        await selectPopoverPage.pressEnterOnOption('apple');
+        await selectPopoverPage.ionPopoverDidDismiss.next();
+        await expect(selectPopoverPage.popover).not.toBeVisible();
+      });
     });
   });
 });
diff --git a/core/src/components/select-popover/test/fixtures.ts b/core/src/components/select-popover/test/fixtures.ts
index 14c81033763..5444a889c04 100644
--- a/core/src/components/select-popover/test/fixtures.ts
+++ b/core/src/components/select-popover/test/fixtures.ts
@@ -63,6 +63,11 @@ export class SelectPopoverPage {
     await option.press('Space');
   }
 
+  async pressEnterOnOption(value: string) {
+    const option = this.getOption(value);
+    await option.press('Enter');
+  }
+
   private getOption(value: string) {
     const { multiple, selectPopover } = this;
     const selector = multiple ? 'ion-checkbox' : 'ion-radio';
diff --git a/core/src/components/select/test/basic/select.e2e.ts b/core/src/components/select/test/basic/select.e2e.ts
index bf23678e657..5a85e3bc2d5 100644
--- a/core/src/components/select/test/basic/select.e2e.ts
+++ b/core/src/components/select/test/basic/select.e2e.ts
@@ -281,6 +281,86 @@ configs({ directions: ['ltr'] }).forEach(({ title, config, screenshot }) => {
         const popover = page.locator('ion-popover');
         await expect(popover).toHaveScreenshot(screenshot(`select-basic-popover-scroll-to-selected`));
       });
+
+      test('opening a popover with Enter should not immediately dismiss it', async ({ page, skip }, testInfo) => {
+        // TODO (ROU-5437)
+        skip.browser('webkit', 'Safari 16 only allows text fields and pop-up menus to be focused.');
+
+        testInfo.annotations.push({
+          type: 'issue',
+          description: 'https://github.com/ionic-team/ionic-framework/issues/30561',
+        });
+
+        await page.setContent(
+          `
+          <ion-app>
+            <ion-select aria-label="Fruit" interface="popover">
+              <ion-select-option value="apple">Apple</ion-select-option>
+              <ion-select-option value="banana">Banana</ion-select-option>
+            </ion-select>
+          </ion-app>
+        `,
+          config
+        );
+
+        const ionPopoverDidPresent = await page.spyOnEvent('ionPopoverDidPresent');
+        const ionPopoverDidDismiss = await page.spyOnEvent('ionPopoverDidDismiss');
+
+        await page.locator('ion-select button').focus();
+        await page.keyboard.press('Enter');
+        await ionPopoverDidPresent.next();
+
+        const popover = page.locator('ion-popover');
+        await expect(popover).toBeVisible();
+
+        await page.waitForChanges();
+        expect(ionPopoverDidDismiss).toHaveReceivedEventTimes(0);
+        await expect(popover).toBeVisible();
+      });
+
+      test('holding Enter to open a popover should not immediately dismiss it', async ({ page, skip }, testInfo) => {
+        // TODO (ROU-5437)
+        skip.browser('webkit', 'Safari 16 only allows text fields and pop-up menus to be focused.');
+
+        testInfo.annotations.push({
+          type: 'issue',
+          description: 'https://github.com/ionic-team/ionic-framework/issues/30561',
+        });
+
+        await page.setContent(
+          `
+          <ion-app>
+            <ion-select aria-label="Fruit" interface="popover">
+              <ion-select-option value="apple">Apple</ion-select-option>
+              <ion-select-option value="banana">Banana</ion-select-option>
+            </ion-select>
+          </ion-app>
+        `,
+          config
+        );
+
+        const ionPopoverDidPresent = await page.spyOnEvent('ionPopoverDidPresent');
+        const ionPopoverDidDismiss = await page.spyOnEvent('ionPopoverDidDismiss');
+        const select = page.locator('ion-select') as E2ELocator;
+        const ionChange = await select.spyOnEvent('ionChange');
+
+        await page.locator('ion-select button').focus();
+        await page.keyboard.down('Enter');
+        await ionPopoverDidPresent.next();
+
+        const popover = page.locator('ion-popover');
+        await expect(popover).toBeVisible();
+
+        // Second down('Enter') fires a repeat keydown (repeat=true),
+        // which is the path guarded against in radio-group.
+        await page.keyboard.down('Enter');
+        await page.keyboard.up('Enter');
+        await page.waitForChanges();
+
+        expect(ionPopoverDidDismiss).toHaveReceivedEventTimes(0);
+        expect(ionChange).toHaveReceivedEventTimes(0);
+        await expect(popover).toBeVisible();
+      });
     });
 
     test.describe('select: modal', () => {
@@ -450,6 +530,49 @@ configs({ modes: ['md'], directions: ['ltr'] }).forEach(({ title, config }) => {
       expect(ionChange).toHaveReceivedEventTimes(1);
     });
 
+    test('should fire ionChange when confirming a popover value with Enter', async ({ page, skip }, testInfo) => {
+      // TODO (ROU-5437)
+      skip.browser('webkit', 'Safari 16 only allows text fields and pop-up menus to be focused.');
+
+      testInfo.annotations.push({
+        type: 'issue',
+        description: 'https://github.com/ionic-team/ionic-framework/issues/30561',
+      });
+
+      await page.setContent(
+        `
+        <ion-app>
+          <ion-select aria-label="Fruit" interface="popover">
+            <ion-select-option value="apple">Apple</ion-select-option>
+            <ion-select-option value="banana">Banana</ion-select-option>
+          </ion-select>
+        </ion-app>
+      `,
+        config
+      );
+
+      const ionPopoverDidPresent = await page.spyOnEvent('ionPopoverDidPresent');
+      const ionPopoverDidDismiss = await page.spyOnEvent('ionPopoverDidDismiss');
+      const select = page.locator('ion-select') as E2ELocator;
+      const ionChange = await select.spyOnEvent('ionChange');
+
+      await select.click();
+      await ionPopoverDidPresent.next();
+
+      const popover = page.locator('ion-popover');
+      const secondRadio = popover.locator('ion-radio').nth(1);
+
+      await secondRadio.focus();
+      await page.keyboard.press('Enter');
+
+      await ionChange.next();
+      await ionPopoverDidDismiss.next();
+
+      expect(ionChange).toHaveReceivedEventDetail({ value: 'banana' });
+      expect(ionChange).toHaveReceivedEventTimes(1);
+      await expect(popover).not.toBeVisible();
+    });
+
     test('should fire ionChange when confirming a value from a popover', async ({ page }) => {
       await page.setContent(
         `

From 7314bbd4df234c4810e82b457f5b34248d44a856 Mon Sep 17 00:00:00 2001
From: ShaneK <shane@shanessite.net>
Date: Thu, 23 Apr 2026 10:10:19 -0700
Subject: [PATCH 2/2] test(select): add keyboard confirmation tests for popover
 and modal interfaces

---
 .../select/test/basic/select.e2e.ts           | 318 ++++++++++++++++++
 1 file changed, 318 insertions(+)

diff --git a/core/src/components/select/test/basic/select.e2e.ts b/core/src/components/select/test/basic/select.e2e.ts
index 5a85e3bc2d5..6b797740b96 100644
--- a/core/src/components/select/test/basic/select.e2e.ts
+++ b/core/src/components/select/test/basic/select.e2e.ts
@@ -601,6 +601,324 @@ configs({ modes: ['md'], directions: ['ltr'] }).forEach(({ title, config }) => {
       expect(ionChange).toHaveReceivedEventTimes(1);
     });
 
+    test('should fire ionChange exactly once when confirming a popover value with Space', async ({
+      page,
+      skip,
+    }, testInfo) => {
+      // TODO (ROU-5437)
+      skip.browser('webkit', 'Safari 16 only allows text fields and pop-up menus to be focused.');
+
+      testInfo.annotations.push({
+        type: 'issue',
+        description: 'https://github.com/ionic-team/ionic-framework/issues/30561',
+      });
+
+      await page.setContent(
+        `
+        <ion-app>
+          <ion-select aria-label="Fruit" interface="popover">
+            <ion-select-option value="apple">Apple</ion-select-option>
+            <ion-select-option value="banana">Banana</ion-select-option>
+          </ion-select>
+        </ion-app>
+      `,
+        config
+      );
+
+      const ionPopoverDidPresent = await page.spyOnEvent('ionPopoverDidPresent');
+      const ionPopoverDidDismiss = await page.spyOnEvent('ionPopoverDidDismiss');
+      const select = page.locator('ion-select') as E2ELocator;
+      const ionChange = await select.spyOnEvent('ionChange');
+
+      await select.click();
+      await ionPopoverDidPresent.next();
+
+      const popover = page.locator('ion-popover');
+      const secondRadio = popover.locator('ion-radio').nth(1);
+
+      await secondRadio.focus();
+      await page.keyboard.press('Space');
+
+      await ionChange.next();
+      await ionPopoverDidDismiss.next();
+
+      expect(ionChange).toHaveReceivedEventDetail({ value: 'banana' });
+      expect(ionChange).toHaveReceivedEventTimes(1);
+      await expect(popover).not.toBeVisible();
+    });
+
+    test('should not fire ionChange when confirming the already-selected popover option with Enter', async ({
+      page,
+      skip,
+    }, testInfo) => {
+      // TODO (ROU-5437)
+      skip.browser('webkit', 'Safari 16 only allows text fields and pop-up menus to be focused.');
+
+      testInfo.annotations.push({
+        type: 'issue',
+        description: 'https://github.com/ionic-team/ionic-framework/issues/26789',
+      });
+
+      await page.setContent(
+        `
+        <ion-app>
+          <ion-select aria-label="Fruit" interface="popover" value="apple">
+            <ion-select-option value="apple">Apple</ion-select-option>
+            <ion-select-option value="banana">Banana</ion-select-option>
+          </ion-select>
+        </ion-app>
+      `,
+        config
+      );
+
+      const ionPopoverDidPresent = await page.spyOnEvent('ionPopoverDidPresent');
+      const ionPopoverDidDismiss = await page.spyOnEvent('ionPopoverDidDismiss');
+      const select = page.locator('ion-select') as E2ELocator;
+      const ionChange = await select.spyOnEvent('ionChange');
+
+      await select.click();
+      await ionPopoverDidPresent.next();
+
+      const popover = page.locator('ion-popover');
+      const selectedRadio = popover.locator('ion-radio').nth(0);
+
+      await selectedRadio.focus();
+      await page.keyboard.press('Enter');
+
+      await ionPopoverDidDismiss.next();
+
+      expect(ionChange).toHaveReceivedEventTimes(0);
+      await expect(popover).not.toBeVisible();
+      await expect(select).toHaveJSProperty('value', 'apple');
+    });
+
+    test('should not fire ionChange when confirming the already-selected popover option with Space', async ({
+      page,
+      skip,
+    }, testInfo) => {
+      // TODO (ROU-5437)
+      skip.browser('webkit', 'Safari 16 only allows text fields and pop-up menus to be focused.');
+
+      testInfo.annotations.push({
+        type: 'issue',
+        description: 'https://github.com/ionic-team/ionic-framework/issues/26789',
+      });
+
+      await page.setContent(
+        `
+        <ion-app>
+          <ion-select aria-label="Fruit" interface="popover" value="apple">
+            <ion-select-option value="apple">Apple</ion-select-option>
+            <ion-select-option value="banana">Banana</ion-select-option>
+          </ion-select>
+        </ion-app>
+      `,
+        config
+      );
+
+      const ionPopoverDidPresent = await page.spyOnEvent('ionPopoverDidPresent');
+      const ionPopoverDidDismiss = await page.spyOnEvent('ionPopoverDidDismiss');
+      const select = page.locator('ion-select') as E2ELocator;
+      const ionChange = await select.spyOnEvent('ionChange');
+
+      await select.click();
+      await ionPopoverDidPresent.next();
+
+      const popover = page.locator('ion-popover');
+      const selectedRadio = popover.locator('ion-radio').nth(0);
+
+      await selectedRadio.focus();
+      await page.keyboard.press('Space');
+
+      await ionPopoverDidDismiss.next();
+
+      expect(ionChange).toHaveReceivedEventTimes(0);
+      await expect(popover).not.toBeVisible();
+      await expect(select).toHaveJSProperty('value', 'apple');
+    });
+
+    test('should fire ionChange exactly once when confirming a modal value with Enter', async ({
+      page,
+      skip,
+    }, testInfo) => {
+      // TODO (ROU-5437)
+      skip.browser('webkit', 'Safari 16 only allows text fields and pop-up menus to be focused.');
+
+      testInfo.annotations.push({
+        type: 'issue',
+        description: 'https://github.com/ionic-team/ionic-framework/issues/30561',
+      });
+
+      await page.setContent(
+        `
+        <ion-app>
+          <ion-select aria-label="Fruit" interface="modal">
+            <ion-select-option value="apple">Apple</ion-select-option>
+            <ion-select-option value="banana">Banana</ion-select-option>
+          </ion-select>
+        </ion-app>
+      `,
+        config
+      );
+
+      const ionModalDidPresent = await page.spyOnEvent('ionModalDidPresent');
+      const ionModalDidDismiss = await page.spyOnEvent('ionModalDidDismiss');
+      const select = page.locator('ion-select') as E2ELocator;
+      const ionChange = await select.spyOnEvent('ionChange');
+
+      await select.click();
+      await ionModalDidPresent.next();
+
+      const modal = page.locator('ion-modal');
+      const secondRadio = modal.locator('ion-radio').nth(1);
+
+      await secondRadio.focus();
+      await page.keyboard.press('Enter');
+
+      await ionChange.next();
+      await ionModalDidDismiss.next();
+
+      expect(ionChange).toHaveReceivedEventDetail({ value: 'banana' });
+      expect(ionChange).toHaveReceivedEventTimes(1);
+      await expect(modal).not.toBeVisible();
+    });
+
+    test('should fire ionChange exactly once when confirming a modal value with Space', async ({
+      page,
+      skip,
+    }, testInfo) => {
+      // TODO (ROU-5437)
+      skip.browser('webkit', 'Safari 16 only allows text fields and pop-up menus to be focused.');
+
+      testInfo.annotations.push({
+        type: 'issue',
+        description: 'https://github.com/ionic-team/ionic-framework/issues/30561',
+      });
+
+      await page.setContent(
+        `
+        <ion-app>
+          <ion-select aria-label="Fruit" interface="modal">
+            <ion-select-option value="apple">Apple</ion-select-option>
+            <ion-select-option value="banana">Banana</ion-select-option>
+          </ion-select>
+        </ion-app>
+      `,
+        config
+      );
+
+      const ionModalDidPresent = await page.spyOnEvent('ionModalDidPresent');
+      const ionModalDidDismiss = await page.spyOnEvent('ionModalDidDismiss');
+      const select = page.locator('ion-select') as E2ELocator;
+      const ionChange = await select.spyOnEvent('ionChange');
+
+      await select.click();
+      await ionModalDidPresent.next();
+
+      const modal = page.locator('ion-modal');
+      const secondRadio = modal.locator('ion-radio').nth(1);
+
+      await secondRadio.focus();
+      await page.keyboard.press('Space');
+
+      await ionChange.next();
+      await ionModalDidDismiss.next();
+
+      expect(ionChange).toHaveReceivedEventDetail({ value: 'banana' });
+      expect(ionChange).toHaveReceivedEventTimes(1);
+      await expect(modal).not.toBeVisible();
+    });
+
+    test('should not fire ionChange when confirming the already-selected modal option with Enter', async ({
+      page,
+      skip,
+    }, testInfo) => {
+      // TODO (ROU-5437)
+      skip.browser('webkit', 'Safari 16 only allows text fields and pop-up menus to be focused.');
+
+      testInfo.annotations.push({
+        type: 'issue',
+        description: 'https://github.com/ionic-team/ionic-framework/issues/26789',
+      });
+
+      await page.setContent(
+        `
+        <ion-app>
+          <ion-select aria-label="Fruit" interface="modal" value="apple">
+            <ion-select-option value="apple">Apple</ion-select-option>
+            <ion-select-option value="banana">Banana</ion-select-option>
+          </ion-select>
+        </ion-app>
+      `,
+        config
+      );
+
+      const ionModalDidPresent = await page.spyOnEvent('ionModalDidPresent');
+      const ionModalDidDismiss = await page.spyOnEvent('ionModalDidDismiss');
+      const select = page.locator('ion-select') as E2ELocator;
+      const ionChange = await select.spyOnEvent('ionChange');
+
+      await select.click();
+      await ionModalDidPresent.next();
+
+      const modal = page.locator('ion-modal');
+      const selectedRadio = modal.locator('ion-radio').nth(0);
+
+      await selectedRadio.focus();
+      await page.keyboard.press('Enter');
+
+      await ionModalDidDismiss.next();
+
+      expect(ionChange).toHaveReceivedEventTimes(0);
+      await expect(modal).not.toBeVisible();
+      await expect(select).toHaveJSProperty('value', 'apple');
+    });
+
+    test('should not fire ionChange when confirming the already-selected modal option with Space', async ({
+      page,
+      skip,
+    }, testInfo) => {
+      // TODO (ROU-5437)
+      skip.browser('webkit', 'Safari 16 only allows text fields and pop-up menus to be focused.');
+
+      testInfo.annotations.push({
+        type: 'issue',
+        description: 'https://github.com/ionic-team/ionic-framework/issues/26789',
+      });
+
+      await page.setContent(
+        `
+        <ion-app>
+          <ion-select aria-label="Fruit" interface="modal" value="apple">
+            <ion-select-option value="apple">Apple</ion-select-option>
+            <ion-select-option value="banana">Banana</ion-select-option>
+          </ion-select>
+        </ion-app>
+      `,
+        config
+      );
+
+      const ionModalDidPresent = await page.spyOnEvent('ionModalDidPresent');
+      const ionModalDidDismiss = await page.spyOnEvent('ionModalDidDismiss');
+      const select = page.locator('ion-select') as E2ELocator;
+      const ionChange = await select.spyOnEvent('ionChange');
+
+      await select.click();
+      await ionModalDidPresent.next();
+
+      const modal = page.locator('ion-modal');
+      const selectedRadio = modal.locator('ion-radio').nth(0);
+
+      await selectedRadio.focus();
+      await page.keyboard.press('Space');
+
+      await ionModalDidDismiss.next();
+
+      expect(ionChange).toHaveReceivedEventTimes(0);
+      await expect(modal).not.toBeVisible();
+      await expect(select).toHaveJSProperty('value', 'apple');
+    });
+
     test('should fire ionChange when confirming multiple values from a popover', async ({ page }) => {
       await page.setContent(
         `