Skip to content

Conversation

@brandyscarney
Copy link
Member

@brandyscarney brandyscarney commented Aug 27, 2025

Issue number: internal


What is the current behavior?

A customTheme object can not change based on the mode.

What is the new behavior?

Adds top level ios and md property checks to flatten with the customTheme object based on the mode.

Does this introduce a breaking change?

  • Yes
  • No

Other information

Requires additional changes in order to test.

@vercel
Copy link

vercel bot commented Aug 27, 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 Aug 29, 2025 5:58pm

@github-actions github-actions bot added the package: core @ionic/core package label Aug 27, 2025
@brandyscarney brandyscarney changed the title refactor(global): update to work with mode specific custom theme overrides feat(themes): update to work with mode specific custom theme overrides Aug 27, 2025
@brandyscarney brandyscarney marked this pull request as ready for review August 27, 2025 21:06
@brandyscarney brandyscarney requested a review from a team as a code owner August 27, 2025 21:06
@brandyscarney brandyscarney requested review from gnbm and removed request for a team August 27, 2025 21:07
@brandyscarney brandyscarney changed the title feat(themes): update to work with mode specific custom theme overrides feat(themes): work with mode specific custom theme overrides Aug 27, 2025
@brandyscarney brandyscarney changed the title feat(themes): work with mode specific custom theme overrides feat(themes): allow mode specific custom theme overrides with modular themes Aug 27, 2025
Base automatically changed from FW-6742 to ionic-modular August 29, 2025 17:53
@brandyscarney brandyscarney merged commit be45f4a into ionic-modular Aug 29, 2025
49 checks passed
@brandyscarney brandyscarney deleted the FW-6743 branch August 29, 2025 18:11
brandyscarney added a commit that referenced this pull request Sep 25, 2025
… themes (#30657)

Issue number: internal

---------

## What is the current behavior?
A `customTheme` object can not change based on the mode.

## What is the new behavior?
Adds top level `ios` and `md` property checks to flatten with the
`customTheme` object based on the mode.

## Does this introduce a breaking change?
- [ ] Yes
- [x] No

## Other information
Requires additional changes in order to test.

---------

Co-authored-by: Brandy Smith <6577830+brandyscarney@users.noreply.github.com>
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.

3 participants