Skip to content

Conversation

@joselrio
Copy link
Contributor

Issue number: resolves #


This is based on #30755 PR to try to clean up things.

What is the current behavior?

  • Select text color was not according design
  • Fill solid type was missing the background color

What is the new behavior?

  • Added the expected color to the select text
  • Added missing background color variable to the fill solid type

Does this introduce a breaking change?

  • Yes
  • No

@joselrio joselrio requested a review from a team as a code owner October 30, 2025 11:04
@joselrio joselrio added package: core @ionic/core package type: bug a confirmed bug report labels Oct 30, 2025
@joselrio joselrio requested a review from gnbm October 30, 2025 11:04
@vercel
Copy link

vercel bot commented Oct 30, 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 Oct 30, 2025 3:29pm

Copy link
Member

@brandyscarney brandyscarney left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one comment about a test regression but otherwise looks good!

@joselrio joselrio merged commit d685d09 into next Oct 30, 2025
49 checks passed
@joselrio joselrio deleted the ROU-12161-2 branch October 30, 2025 19:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

package: core @ionic/core package type: bug a confirmed bug report

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants