Skip to content

test(scripts): default the test page theme to the mode - #31672

Open
thetaPC wants to merge 1 commit into
ionic-modularfrom
test-page-theme-default
Open

thetaPC wants to merge 1 commit into
ionic-modularfrom
test-page-theme-default

Conversation

@thetaPC

@thetaPC thetaPC commented Oct 29, 2026

Copy link
Copy Markdown
Contributor

Issue number: internal


What is the current behavior?

A test page opened with only ionic:mode loads the md tokens, even though the app sets the theme from the mode. For example, ?ionic:mode=ios renders ios components with md token values, such as an 8px chip radius instead of 10px.

What is the new behavior?

  • Test pages default the theme to the mode, as the app does, then to md. For example, ?ionic:mode=ios without ionic:theme now renders with the ios theme. An explicit ionic:theme still wins.
  • Screenshots don't change: Playwright always passes the theme. This only affects pages opened by hand and the Vercel previews.

Does this introduce a breaking change?

  • Yes
  • No

@vercel

vercel Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
ionic-framework Ready Ready Preview Oct 29, 2026 10:21pm UTC

Request Review

@github-actions github-actions Bot added the package: core @ionic/core package label Oct 29, 2026
@thetaPC
thetaPC marked this pull request as ready for review October 29, 2026 22:41
@thetaPC
thetaPC requested a review from a team as a code owner October 29, 2026 22:41
@thetaPC
thetaPC requested a review from brandyscarney October 29, 2026 22:41

This branch was successfully deployed

1 active deployment
Preview — 146d068c Deployed Oct 29, 2026 by vercel[bot]
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.

1 participant