Merged
Conversation
削除はしたけれど、このGithubに残り続ける
nkoji21
added
♻️ リファクタリング
There was a problem hiding this comment.
Pull Request Overview
このPRは、プロジェクトのコード整理と構造改善を目的とした大規模なリファクタリングです。使用されていないファイルやコンポーネントの削除、ディレクトリ構造の最適化、型定義の統合を行っています。
- App.tsxからルーティング実装をAppRoutes.tsxに分離し、簡素化
- 未使用のgame/フォルダとcamera/フォルダの整理
- グローバルCSSをstylesフォルダに集約
- 型定義の統合と重複削除
Reviewed Changes
Copilot reviewed 22 out of 22 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| apps/web/wrangler.jsonc | 環境変数設定の削除 |
| apps/web/src/types/index.ts | CameraRef型の追加 |
| apps/web/src/styles/index.css | ピクセルアート関連CSSの削除 |
| apps/web/src/styles/App.module.css | App.module.cssをstylesフォルダに移動 |
| apps/web/src/main.tsx | インポートパスの調整 |
| apps/web/src/components/result/ResultScreen.tsx | インポートパスの調整とコメント削除 |
| apps/web/src/components/photo/PhotoScreen.tsx | インポートパスとtype importの統合 |
| apps/web/src/components/photo/PhotoPreview.tsx | インポートパスの調整 |
| apps/web/src/components/mode/ModeScreen.tsx | インポートパスの調整 |
| apps/web/src/components/index.tsx | コンポーネントエクスポートファイルの削除 |
| apps/web/src/components/home/HomeScreen.tsx | インポートパスの調整 |
| apps/web/src/components/game/types.ts | 未使用型定義ファイルの削除 |
| apps/web/src/components/game/timer.tsx | 未使用タイマーコンポーネントの削除 |
| apps/web/src/components/game/timer.module.css | 未使用CSSファイルの削除 |
| apps/web/src/components/game/shooting-screen.tsx | 未使用撮影画面コンポーネントの削除 |
| apps/web/src/components/game/shooting-screen.module.css | 未使用CSSファイルの削除 |
| apps/web/src/components/camera/types.ts | カメラ型定義ファイルの削除 |
| apps/web/src/components/camera/index.tsx | カメラエクスポートファイルの削除 |
| apps/web/src/components/camera/camera.tsx | 型定義のローカル化とimport統合 |
| apps/web/src/AppRoutes.tsx | ルーティング定義の新規作成 |
| apps/web/src/App.tsx | ルーティング実装の分離と簡素化 |
| apps/web/src/App.module.css | ファイルの削除とstylesフォルダへの移動 |
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.
📝 変更内容
何を変更したか
App.tsxに書いていた各ルーティングの実装を、AppRoutes.tsxに分けることで、簡素化。camera/game/の整理styles/を作成し、そこにグローバルCSSである、App.module.cssindex.cssを格納そのファイルでしか使用していない、型定義については export せず、そのコンポーネントの上部に配置するよう調整
なぜ変更したか
🧪 テスト・確認項目
動作確認
📸 スクリーンショット（UI変更がある場合）
💡 補足事項
🔗 関連Issue
Closes #79