-
Notifications
You must be signed in to change notification settings - Fork 2
♻️ refactor: 모바일 반응형 개선 #124
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Walkthrough여러 컴포넌트와 페이지에서 CSS 클래스명이 수정되어, 반응형 스타일링과 레이아웃이 조정되었습니다. 주요 변경점은 모바일 및 태블릿 와이드 브레이크포인트에 맞춘 크기, 간격, 패딩, 정렬 방식 등 스타일 속성의 업데이트입니다. 로직이나 구조에는 변화가 없습니다. Changes
Possibly related PRs
Suggested labels
Poem
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
npm error Exit handler never called! ✨ Finishing Touches
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (1)
src/app/features/landing/components/Footer.tsx (1)
8-8: 모바일 텍스트 크기 접근성 검토 제안
mobile-wide:text-12이 작은 텍스트 크기여서 가독성에 영향을 줄 수 있습니다. 프로젝트 접근성 가이드라인과 일치하는지 검토해주세요.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (5)
src/app/features/auth/components/AuthLogo.tsx(1 hunks)src/app/features/landing/components/Footer.tsx(1 hunks)src/app/features/mypage/components/ProfileImageUpload.tsx(1 hunks)src/app/mypage/layout.tsx(1 hunks)src/app/mypage/page.tsx(1 hunks)
🔇 Additional comments (5)
src/app/features/landing/components/Footer.tsx (1)
7-7: 클래스 유틸리티 정의 여부 확인 요청
px-140,gap-30,h-236같은 커스텀 스페이싱 유틸리티가 tailwind.config.js에 제대로 정의되어 있는지 확인해주세요.src/app/mypage/layout.tsx (1)
14-14: padding-left 클래스 순서 및 정의 확인 요청
pl-300,mobile-wide:pl-67,tablet-wide:pl-160의 순서가 의도한 mobile-first 설계와 일치하는지, 그리고 해당 값들이 Tailwind 설정에 등록돼 있는지 점검이 필요합니다.src/app/mypage/page.tsx (1)
10-10: 브레이크포인트 적용 우선순위 검토
p-20 mobile-wide:p-10조합이 예상한 화면 크기별 패딩 값 변화를 정확히 반영하는지 확인해주세요. mobile-wide가 min-width인지 max-width인지에 따라 동작이 달라질 수 있습니다.src/app/features/auth/components/AuthLogo.tsx (1)
17-18: 커스텀 width/height 유틸리티 검증 요청
w-230,h-250→mobile-wide:h-150로 변경된 클래스들이 Tailwind 설정에 존재하는지 확인해주세요. 클래스가 누락되면 로고가 의도와 다르게 렌더링될 수 있습니다.src/app/features/mypage/components/ProfileImageUpload.tsx (1)
86-86: 컨테이너 크기 유틸리티 정의 확인
size-182,basis-182,mobile-wide:size-140,mobile-wide:basis-140커스텀 유틸리티가 tailwind.config.js에 반영되어 있는지 검토해주세요.
📌 변경 사항 개요
✨ 요약
mobile-wide적용✅ 체크리스트
💡 참고 사항
Summary by CodeRabbit