refactor: use shared theme utilities - #172
Conversation
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
| }, | ||
| "homepage": "https://docs.rdlabo.dev/projects/ionic-theme-ios27", | ||
| "dependencies": { | ||
| "@rdlabo/ionic-theme-utils": "git+ssh://git@github.com/rdlabo-dev/ionic-theme-utils.git#main" |
There was a problem hiding this comment.
🔴 公開 npm パッケージを通常の利用者がインストールできない Git 依存
依存取得に GitHub の SSH 認証が必須となり、SSH 鍵のない npm 利用者はインストールできません。さらに #main の更新後は同じリリースでも異なるコードを取得します。
Learn more
このプロジェクトの lockfile は現在の開発環境だけをコミット 9e887c0… に固定します。公開 tarball 内の package.json は git+ssh://…#main のままなので、利用者のインストールでは lockfile が使われません。GitHub の SSH URL は公開リポジトリでも利用者側の SSH 認証を要求します。また、ブランチ参照は後から別コミットを指すため、既存リリースの依存内容まで変化します。
Example: SSH 鍵を登録していない利用者が npm install @rdlabo/ionic-theme-ios27 を実行すると、依存リポジトリの clone が認証で失敗します。鍵がある利用者でも、main 更新の前後で同じ @rdlabo/ionic-theme-ios27@0.1.0 から異なる utility コードが導入されます。
Recommended fix: @rdlabo/ionic-theme-utils を npm に公開し、固定バージョンへ依存してください。Git 依存を継続する場合は、認証不要な HTTPS URLと不変のタグまたはコミット SHA を指定してください。
Was this helpful? React with 👍 or 👎 to provide feedback.
| export const iosTransitionAnimation: (navEl: HTMLElement, opts: TransitionOptions) => Animation = | ||
| createIosTransitionAnimation<TransitionOptions>({ | ||
| offLeftPercent: 30, | ||
| getIonPageElement, |
There was a problem hiding this comment.
🟡 iOS 27 の戻り側ページ移動量が縮小する
共有トランジションへ渡す値が従来の 33% から 30% に変わります。前進・戻り遷移で背景ページとツールバーの移動量が小さくなります。
| getIonPageElement, | |
| offLeftPercent: 33, |
Was this helpful? React with 👍 or 👎 to provide feedback.
| createIosTransitionAnimation<TransitionOptions>({ | ||
| offLeftPercent: 30, | ||
| getIonPageElement, | ||
| connectNativeUIShellTransition, | ||
| }); |
There was a problem hiding this comment.
🔍 遷移シェードの DOM クラス名変更
共有 utility は ios27-transition-shade を ios-transition-shade に変更します。外部 CSS やテストが旧名へ依存していないか確認してください。
Was this helpful? React with 👍 or 👎 to provide feedback.
Playwright test results
Details
|
|
/update-screenshots |
|
❌ Failed to update screenshots. Please check the workflow run for details. |
|
/update-screenshots |
|
📊 Ionic 9 Playwright Test Report View the detailed Ionic 9 report: https://rdlabo-dev.github.io/ionic-theme-ios27/pr-172/ Ionic 8 runs against the same screenshots in a separate matrix job. View both results in the workflow run. |
|
ℹ️ No screenshot changes detected. The current screenshots are already up to date. |
npm beta publishedCI passed for the merge commit npm install @rdlabo/ionic-theme-ios27@0.1.0-beta.pr172.sha0316ffe45969 |
1 participant
Summary
Verification