Skip to content

Conversation

@OS-martacarlos
Copy link

Issue number: resolves internal


What is the current behavior?

When opening a ion-select with a modal interface, the first option is focused and, as such, has a border.
This happens because an ion-item is being rendered, which has a specific styling for focused stated.

What is the new behavior?

When inside a ion-select-modal, we don't want this specific ion-item styling.

Does this introduce a breaking change?

  • Yes
  • No

@OS-martacarlos OS-martacarlos self-assigned this Sep 1, 2025
@OS-martacarlos OS-martacarlos requested a review from a team as a code owner September 1, 2025 16:41
@OS-martacarlos OS-martacarlos added the package: core @ionic/core package label Sep 1, 2025
@vercel
Copy link

vercel bot commented Sep 1, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
ionic-framework Ready Ready Preview Comment Sep 1, 2025 4:50pm

@OS-martacarlos OS-martacarlos changed the title fix(select-modal): hide focus option's border for ionic theme fix: hide outlines for select with modal interface for ionic theme Sep 1, 2025
@OS-martacarlos OS-martacarlos changed the title fix: hide outlines for select with modal interface for ionic theme fix(select-modal): hide focus option's border for ionic theme Sep 1, 2025
@OS-martacarlos OS-martacarlos merged commit b2f4d4a into next Sep 3, 2025
91 of 106 checks passed
@OS-martacarlos OS-martacarlos deleted the ROU-12158 branch September 3, 2025 08:07
OS-martacarlos added a commit that referenced this pull request Sep 3, 2025
Issue number: resolves internal

---------
## What is the current behavior?
When opening a ion-select with a modal interface, the first option is
focused and, as such, has a border.
This happens because an ion-item is being rendered, which has a specific
styling for focused stated.

## What is the new behavior?
When inside a `ion-select-modal`, we don't want this specific `ion-item`
styling.

## Does this introduce a breaking change?
- [ ] Yes
- [x] No
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

package: core @ionic/core package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants