Skip to content

全画面のモバイルサイズのデザイン調整#71

Merged
miyakoshi-3854 merged 8 commits intomainfrom
feature/63
Jul 27, 2025
Merged

全画面のモバイルサイズのデザイン調整#71
miyakoshi-3854 merged 8 commits intomainfrom
feature/63

Conversation

@nkoji21
Copy link
Copy Markdown
Member

@nkoji21 nkoji21 commented Jul 27, 2025

📝 変更内容

何を変更したか

なぜ変更したか


🧪 テスト・確認項目

動作確認

  • プルリクエストにラベルを追加したか
  • アサインに自分を追加したか
  • ローカル環境で動作確認済み
  • 既存機能に影響がないことを確認
  • モバイル表示を確認(該当する場合)

📸 スクリーンショット(UI変更がある場合)

Before After
Before After
localhost_5173_ localhost_5173_ (1) localhost_5173_photo localhost_5173_photo (1) localhost_5173_photo (2)

💡 補足事項


🔗 関連Issue

Closes #

@nkoji21 nkoji21 self-assigned this Jul 27, 2025
Copilot AI review requested due to automatic review settings July 27, 2025 04:57
@nkoji21 nkoji21 added the 🎨 UI改善 見た目やデザイン、使いやすさに関するタスク label Jul 27, 2025
@nkoji21 nkoji21 requested a review from miyakoshi-3854 as a code owner July 27, 2025 04:57
@nkoji21 nkoji21 added 🖥️ フロントエンド ユーザーが直接見る画面や操作に関わるタスク 🔥 優先度: 高 最も優先度が高い。すぐに取り掛かるべきタスク 🔧 機能修正 既存の機能の挙動を修正・変更するタスク labels Jul 27, 2025
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Mobile-first responsive design adjustments for fullscreen mobile layout. The changes extract common screen styles to a global CSS module and implement responsive breakpoints to optimize the interface for mobile devices with screens smaller than 460px.

  • Centralized screen styling by extracting common .screen class to App.module.css
  • Added responsive design breakpoints for mobile devices (max-width: 460px)
  • Implemented fullscreen mobile layout with removed border-radius and adjusted container dimensions

Reviewed Changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated no comments.

Show a summary per file
File Description
apps/web/src/App.module.css Added centralized .screen class and mobile responsive breakpoints for fullscreen layout
apps/web/src/components/home/HomeScreen.tsx Updated to use centralized screen class from App.module.css
apps/web/src/components/home/HomeScreen.module.css Removed duplicate screen styles and adjusted spacing for mobile
apps/web/src/components/mode/ModeScreen.tsx Updated to use centralized screen class from App.module.css
apps/web/src/components/mode/ModeScreen.module.css Removed duplicate screen styles and added mobile-specific positioning
apps/web/src/components/photo/PhotoScreen.tsx Updated to use centralized screen class from App.module.css
apps/web/src/components/photo/PhotoScreen.module.css Removed duplicate screen styles and optimized camera layout for mobile
apps/web/src/components/photo/PhotoPreview.tsx Updated to use centralized screen class from App.module.css
apps/web/src/components/photo/PhotoPreview.module.css Removed duplicate screen styles and adjusted photo preview dimensions for mobile
apps/web/src/components/result/ResultScreen.tsx Updated to use centralized screen class from App.module.css
apps/web/src/components/result/ResultScreen.module.css Removed duplicate screen styles
Comments suppressed due to low confidence (1)

apps/web/src/components/photo/PhotoScreen.module.css:15

  • Class name changed from 'photo-Content' to 'photo-content' but the corresponding usage in the JSX file is not visible in this diff. Ensure the class name is updated consistently in the PhotoScreen.tsx component.
.photo-content {

Copy link
Copy Markdown
Collaborator

@miyakoshi-3854 miyakoshi-3854 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@miyakoshi-3854 miyakoshi-3854 merged commit a5b6c25 into main Jul 27, 2025
1 check passed
@miyakoshi-3854 miyakoshi-3854 deleted the feature/63 branch July 27, 2025 04:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🎨 UI改善 見た目やデザイン、使いやすさに関するタスク 🔥 優先度: 高 最も優先度が高い。すぐに取り掛かるべきタスク 🔧 機能修正 既存の機能の挙動を修正・変更するタスク 🖥️ フロントエンド ユーザーが直接見る画面や操作に関わるタスク

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants