-
Notifications
You must be signed in to change notification settings - Fork 0
[fix] 메인페이지 UI 개선 #6
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
base: main
Are you sure you want to change the base?
Conversation
- 48px -> 40px로 변경
- margin-vertical은 위아래 margin이므로 반으로 줄임
- 탭 바 높이 조정 및 라벨 표시 (Show Label) - 탭 아이콘과 라벨 사이 간격 및 스타일 조정 - 홈 및 메뉴 탭 아이콘을 PNG에서 SVG로 교체하여 색상 제어 가능하도록 개선 - 비활성 탭 색상 통일
- #687076 -> #C5C5C5
요약탭 레이아웃 재구성, Mixpanel 이벤트 추적 추가, 테마 색상 업데이트, 그리고 홈 화면 컴포넌트 전반의 패딩과 스타일 조정이 이루어졌습니다. 변경 사항
예상 코드 리뷰 노력🎯 3 (중간) | ⏱️ ~20분 🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
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
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
app/(tabs)/_layout.tsx (1)
16-21: 홈 탭 클릭 트래킹 URL이 실제 라우트와 불일치합니다.홈 탭의 screen name은
index인데handleTabPress("home")로 전달되어 URL이/home으로 기록됩니다. 라우트명과 표시명을 분리해 기록하는 방식이 안전합니다.🔧 수정 제안
- const handleTabPress = (tabName: string) => { + const handleTabPress = (tabName: string, routeName: string = tabName) => { trackEvent(USER_EVENT.BOTTOM_TAB_CLICKED, { tab: tabName, - url: `app://moadong/(tabs)/${tabName}`, + url: `app://moadong/(tabs)/${routeName}`, }); };- listeners={{ - tabPress: () => handleTabPress("home"), - }} + listeners={{ + tabPress: () => handleTabPress("home", "index"), + }}Also applies to: 81-83
변경사항
작업내용
1. 하단 탭 바 UI 개선
tabBarShowLabel: true)을 활성화하여 라벨 보이도록 변경알림 | 홈 | 메뉴=>메뉴 | 홈 | 알림2. 메인 헤더 스타일 수정
theme.ts의theme.Colors.light에서 하단 탭바 아이콘과 글자 색상에 맞춰 color를 변경했습니다.theme.Colors.light를 가져다 쓰던 검색창의 placeholder와 icon의 색상과 불일치하여 인라인에서 직접 color를 적용했습니다.3. 메인 UI 자잘한 수정
✨ 리뷰받고 싶은 부분
하단 탑바 아이콘 위치를 변경했는데 기존의 아이콘 위치 설정에서 의도가 있었는지 궁금합니다.
Summary by CodeRabbit
릴리스 노트
새로운 기능
개선사항
✏️ Tip: You can customize this high-level summary in your review settings.