Skip to content

fix: avoid installing Ionic for plain HTML Native UI consumers - #310

Merged
rdlabo merged 1 commit into
mainfrom
fix/plain-html-optional-ionic
Oct 20, 2026
Merged

rdlabo merged 1 commit into
mainfrom
fix/plain-html-optional-ionic

Conversation

@rdlabo

@rdlabo rdlabo commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

Plain HTML Native UI consumers currently install Ionic transitively through utils, despite Native UI declaring Ionic optional. Update iOS 27, Native UI, and the root build dependency to published @rdlabo/ionic-theme-utils@0.1.5, whose Ionic peer is optional. The theme still requires Ionic, and shared animation code stays in utils.

Changes are limited to dependency pins and root/demo lockfiles.

Fixes #306. Uses rdlabo-dev/ionic-theme-utils#11.

Validation

  • Build, ESM/package checks, and lint passed.
  • Demo clean install and all 161 tests passed.
  • A fresh consumer installed packed Native UI with published utils 0.1.5 without Ionic, Stencil, or ionicons; the /core import passed.
  • After explicitly installing Ionic, the default Native UI adapter and shared popover animation imports passed.

@github-actions

Copy link
Copy Markdown
Contributor

Playwright test results

passed  454 passed

Details

stats  454 tests across 24 suites
duration  3 minutes, 40 seconds
commit  bc48791
info  This detailed result covers Ionic 9 only. Ionic 8 runs against the same screenshots in a separate matrix job; check the workflow run for both results. To update the screenshots, comment with /update-screenshots.

@github-actions

Copy link
Copy Markdown
Contributor

📊 Ionic 9 Playwright Test Report

View the detailed Ionic 9 report after GitHub Pages deployment completes: https://rdlabo-dev.github.io/ionic-theme-ios27/pr-310/

Ionic 8 runs against the same screenshots in a separate matrix job. View both results in the workflow run.

@rdlabo
rdlabo merged commit b88b795 into main Oct 20, 2026
13 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

npm beta published

CI passed for the merge commit b88b79526a19. Install the immutable version with:

npm install @rdlabo/ionic-theme-ios27@2.0.0-beta.pr310.shab88b79526a19 @rdlabo/capacitor-native-ui@2.0.0-beta.pr310.shab88b79526a19

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Avoid installing Ionic transitively for plain HTML Native UI consumers

1 participant