Repository navigation
fix: iOS版を明るい配色に固定し、起動画面から編集画面までの色をそろえる - #118
Merged
Merged
Conversation
Fix the iOS app to the light appearance for v1.0, match the startup loading overlay to the launch screen color, and give the in-app loading overlay and the WKWebView background the editor page color, so dark mode no longer shows a black screen between the launch screen and the editor. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
K0mork
commented
Oct 6, 2026
K0mork
left a comment
Owner
Author
There was a problem hiding this comment.
レビュー結果
CI
実行 37458945971 のジョブはすべて成功しました(changes、app_store_docs、ios_web、ios(iPhone 17・iPad (A16))、app_update(iPhone 17・iPad (A16))、release_archive、ci-gate)。CodeQL も成功です。web、deploy、smoke は iOS だけの変更なのでスキップされています。両方の ios ジョブで、追加した AppAppearanceTests 7件が実行され、すべて成功したことをログで確認しました。
確認したこと(問題なし)
- 色の流れ: 起動直後は
LaunchBackground、webReadyのあとはhasShownEditorによって#f3f0e8になります。使い方ページ(guide/index.htmlの地の色も#f3f0e8)から戻るときに緑がはさまらないことも、コードとテストで確かめました。WKWebViewとscrollViewの背景も#f3f0e8で、base.cssの:rootと一致しています。 UIUserInterfaceStyle=Lightは Issue の選択肢「明るい配色に固定する」と合っています。ダークモードへの対応は #116 に分けてあります。.knitの受け渡しへの影響:webContentReady、pendingBackup、isPreparingEditorの扱いは変わっていません。足したのは配色を選ぶフラグだけなので、影響は無いと判断しました。project.pbxproj: PR の内容からxcodegen generateで作り直した結果と完全に一致しました。docs/dev-log/のエントリは README の書式どおりです。手元で実行した確認と、PR の CI に任せた確認(Simulator のテスト一式、アプリ更新テスト、Release Archive、同梱物の検査)が分けて書かれています。- 差分、PR 本文、コミットメッセージに機密情報はありませんでした。
Closes #79とbugラベルもあります。 - 最新の
mainから2コミット遅れていますが、コンフリクトはありません。
指摘(どれも軽微で、マージを止めるものではありません)
- (軽微・誤記)
ios/App/KnittingEditorApp.swiftのAppColors.editorSecondaryTextのドキュメントコメントが編集画面の補足の文字の色()。となっていて、括弧の中が空です。参照元(base.cssの.gesture-hintのcolorなど)を書くか、括弧を消してください。 - (任意)
testEditorPageBackgroundMatchesBundledStylesheetは、圧縮後の CSS の書き方background:#f3f0e8を文字列のまま探しています。CSS の圧縮方法や宣言の書き方が変わると、色が同じでも失敗します。今は CI で成功しているので、このままでも構いません。
Issue #79 の完了条件と規約のうち、満たしていないもの
- PR に画面の記録が添付されていません。 Issue は「iPhone と iPad の Simulator で、ライト・ダークそれぞれの起動から編集画面までを画面収録する」ことを求めています。確認は手元で行われていますが、収録もスクリーンショットも PR に付いていません。AGENTS.md の「UI の変更ではスクリーンショットを PR に含める」にも当たるので、ライト・ダークのそれぞれ、iPhone と iPad について、起動画面→準備中の表示→編集画面のスクリーンショット(または収録)を添付してください。
- 使い方ページから戻るときの準備中の表示(
#f3f0e8)は、画面では確認されていません(単体テストだけ)。PR 本文に書かれているとおりです。 - 実機(TestFlight)での確認は、Issue の完了条件には含まれていません(
needs-deviceもありません)。デプロイ影響の欄に、配信後に確かめることとして書かれています。
… string - Fill in the empty source reference in the AppColors.editorSecondaryText doc comment. - Extract the :root background color from the bundled CSS with a whitespace-tolerant regex and compare it with AppColors.editorPageBackground. - Record the review follow-up and the Simulator check of returning from the guide page. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Owner
Author
レビュー指摘への対応(47862da)
確認
残っていること
|
This was referenced Oct 7, 2026
Owner
Author
デプロイ確認
途中の このPRについてこのPRは iOS だけの変更なので、Pages の配信には影響しません。この run で iOS のジョブ( |
This was referenced Oct 7, 2026
K0mork
added a commit
that referenced
this pull request
Oct 7, 2026
Keep both sides in WebViewContainer.swift: the launch colors and loading style from #118, and the menu command state and native UI flag from #125. The share sheet completion now clears the native UI flag and reports the finished export for the review request (#121). Regenerate project.pbxproj with XcodeGen. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
変更内容
端末がダークモードのとき、起動画面(緑)→準備中の表示(黒)→編集画面(明るい生成り色)と色が切り替わっていたのを直します。ライトモードでも準備中の表示は白でした。
ios/App/Info.plistにUIUserInterfaceStyle=Lightを足し、iOS版を明るい配色に固定しました。準備中の表示、保存画面などのシステムのダイアログ、ステータスバーの領域も明るい配色になります。LaunchBackgroundの緑にしました。文字とくるくる回る表示は白です(コントラスト比は約6:1)。#f3f0e8にしました(文字は#526059、約5.9:1)。アプリの中で緑が一瞬出ることはありません。WKWebViewの背景を.systemBackgroundから#f3f0e8に変えました。文書が描かれるまでの間に白や黒が見えなくなります。packages/も変えていません。決めたこと
実行した確認
xcodegen generate --spec ios/project.yml: 新しいテストファイルがproject.pbxprojに入ることを確認しました。xcodebuild test -project ios/knittingEditor.xcodeproj -scheme knittingEditor -destination 'platform=iOS Simulator,id=<udid>' -only-testing:knittingEditorTests CODE_SIGNING_ALLOWED=NOを実行: 33件中、機内モード用の1件がskip、失敗0件(追加したAppAppearanceTestsは7件すべて成功)。変更したファイルで警告は増えていません。xcrun simctl io recordVideoの画面収録とxcrun simctl io screenshotの連続撮影で、ライト・ダークそれぞれの起動から編集画面までを確かめました(収録はリポジトリに入れていません)。端末は iPhone 18 Pro(iOS 27.0)、iPhone 16(iOS 18.2)、iPad (10th generation)(iOS 18.2)です。packages/は変えていないので、Webの必須スイートとiOS Webの検査は実行していません。追加したテスト(
ios/Tests/KnittingEditorAppTests/AppAppearanceTests.swift):UIUserInterfaceStyleがLightであるLaunchBackgroundを使い、起動直後の準備中の表示が同じ色である(ライト・ダークの両方で)WKWebViewの背景が#f3f0e8で、同梱したCSSにも同じ色がある残っている懸念(#117 に分けました)
webReadyが端末内データの準備より前に届くためで、この変更の前からあります。消す時期を変えると.knitの受け渡しや初期化失敗時の表示に影響するので、このPRでは変えていません。display-p3にしても差は残ったので、色定義は変えていません。実機で確かめる必要があります。デプロイ影響
Pagesへの配信はありません(iOSだけの変更)。TestFlightに載せたら、実機のライト・ダークそれぞれで、起動から編集画面まで色がちらつかないことを確かめます。
Closes #79
🤖 Generated with Claude Code