Skip to content

Conversation

@brandyscarney
Copy link
Member

@brandyscarney brandyscarney commented Aug 22, 2025

Issue number: internal


What is the new behavior?

  • Moves openURL out of the theme utils because it makes more sense in helpers
  • Adds support for the default default.tokens.ts design tokens file
  • Adds support for custom theme set globally and on a component

Does this introduce a breaking change?

  • Yes
  • No

Other information

Requires additional changes in order to test.

@vercel
Copy link

vercel bot commented Aug 22, 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 28, 2025 8:15pm

@brandyscarney brandyscarney marked this pull request as ready for review August 25, 2025 19:48
@brandyscarney brandyscarney requested a review from a team as a code owner August 25, 2025 19:48
@brandyscarney brandyscarney requested a review from ShaneK August 25, 2025 19:48
Copy link
Member

@ShaneK ShaneK left a comment

Choose a reason for hiding this comment

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

This is looking really good, I tested it and everything seems to be working well. Great work!

@brandyscarney brandyscarney merged commit cca089f into ionic-modular Aug 29, 2025
78 of 94 checks passed
@brandyscarney brandyscarney deleted the FW-6742 branch August 29, 2025 17:53
brandyscarney added a commit that referenced this pull request Sep 25, 2025
Issue number: internal

---------

## What is the new behavior?
- Moves `openURL` out of the `theme` utils because it makes more sense
in `helpers`
- Adds support for the default `default.tokens.ts` design tokens file
- Adds support for custom theme set globally and on a component 

## Does this introduce a breaking change?

- [ ] Yes
- [x] No

## Other information

Requires additional changes in order to test.

---------

Co-authored-by: Brandy Smith <[email protected]>
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