Skip to content

Conversation

@brandyscarney
Copy link
Member

@brandyscarney brandyscarney commented Nov 5, 2025

Issue number: N/A


What is the current behavior?

The Datetime component breaks CSP rules due to the following:

  1. Datetime is using ion-buttons which uses scoped encapsulation.
  2. When using the ionic theme, it imports Phosphor icons as data URIs, which are blocked by connect-src 'self' blob:.

What is the new behavior?

  • Remove all usages of ion-buttons, removing the dependency on a scoped component
  • Remove all references to Phosphor icons
  • Updates the styling to account for the removal of ion-buttons

Does this introduce a breaking change?

  • Yes
  • No

@vercel
Copy link

vercel bot commented Nov 5, 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 Nov 6, 2025 2:23pm

@github-actions github-actions bot added the package: core @ionic/core package label Nov 5, 2025
Copy link
Member Author

Choose a reason for hiding this comment

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

I can't reproduce this shift locally which leads me to believe it's just a rendering difference.

Copy link
Member Author

Choose a reason for hiding this comment

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

All ionic theme diffs are due to the arrow icons changing.

Copy link
Member Author

@brandyscarney brandyscarney Nov 5, 2025

Choose a reason for hiding this comment

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

I can't see any difference locally which leads me to believe it's just a rendering difference.

Copy link
Member Author

Choose a reason for hiding this comment

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

I can't see any difference locally which leads me to believe it's just a rendering difference.

Copy link
Member Author

Choose a reason for hiding this comment

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

I can't reproduce this shift locally which leads me to believe it's just a rendering difference.

Copy link
Member Author

Choose a reason for hiding this comment

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

I can't reproduce this shift locally which leads me to believe it's just a rendering difference.

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