style: UI 일관성 정리 및 입력 다이얼로그 엔터 제출 지원 - #413
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Discord 스레드 연동용 메타데이터입니다. discord-pr-bot 워크플로가 자동 생성하며, 수정·삭제하면 PR 과 Discord 알림 연동이 끊깁니다. |
|
Warning Review limit reached
Next review available in: 21 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (18)
📝 WalkthroughWalkthrough여러 다이얼로그의 제출 처리를 폼 이벤트 기반으로 변경했습니다. 법률, 알림, 마이페이지, 토너먼트 헤더의 제목 스타일을 조정했습니다. 마이페이지 상단 구조와 Wish 카드 로딩 UI를 정리했습니다. Changes폼 제출 처리
페이지 UI 업데이트
Wish 로딩 UI
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested labels: Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 Generate unit tests (beta)
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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@apps/web/src/app/mypage/page.tsx`:
- Around line 20-23: 페이지의 바깥 컨테이너를 semantic <main>으로 변경해 `h1`과 `Spacing`을 동일한 주요
콘텐츠 영역에 포함하세요. 현재 내부 `<main>`은 스크롤 동작을 유지하는 `<div>`로 바꾸고, 제목은 고정된 상태에서 본문만
스크롤되도록 기존 클래스와 레이아웃을 보존하세요.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 65a90b42-1225-4e00-8af2-c581be28e1f9
📒 Files selected for processing (14)
apps/web/src/app/(legal)/privacy/page.tsxapps/web/src/app/(legal)/terms/page.tsxapps/web/src/app/archive/wish/_components/WishCardSkeleton.tsxapps/web/src/app/home/_components/CreateTournamentDialog.tsxapps/web/src/app/home/_components/InviteTournamentDialog.tsxapps/web/src/app/mypage/_components/ProfileSection.tsxapps/web/src/app/mypage/edit/page.tsxapps/web/src/app/mypage/page.tsxapps/web/src/app/mypage/withdraw/page.tsxapps/web/src/app/notification/_components/NotificationContent.tsxapps/web/src/app/tournament/[id]/create/_components/tournament-header/TournamentHeader.tsxapps/web/src/components/common/create-tournament-dialog/index.tsxapps/web/src/components/common/wish-card/index.tsxapps/web/src/components/get-item-dialog/ByLinkDialog.tsx
💤 Files with no reviewable changes (1)
- apps/web/src/app/mypage/_components/ProfileSection.tsx
68d34b1 to
f72860c
Compare
1 participant
작업 요약
작업 내용
UI 일관성이 어긋나 있던 부분들을 시안 기준으로 정리하고, 입력 다이얼로그의 사용성을 개선했습니다.
1. 헤더 타이틀 크기 통일 (24px → 20px)
Header의 중앙 타이틀이 화면마다title-1(24px)과heading-1-bold(20px)로 갈려 있어heading-1-bold로 통일했습니다.TournamentHeader는 폭·정렬 클래스(w-[calc(...)] text-center)를 그대로 두고 폰트 클래스만 교체했습니다.매치 화면 질문 문구와 온보딩 문구는 헤더가 아닌 본문 제목이라
title-1을 유지했습니다.2. 마이페이지 헤더를 위시리스트와 동일한 형태로
중앙 정렬 헤더 + 뒤로가기 버튼 구조를, 위시리스트(
내 위시리스트)와 같은 좌측 정렬h1으로 변경했습니다.설정→내 정보로 변경 — 알림·앱 설정 같은 토글 항목 없이 프로필·계정 관리 위주라 실제 내용과 더 맞음프로필소제목 제거3. 위시 상품 카드 스켈레톤 라운드 제거
실제
WishCard에는 라운드가 없는데 스켈레톤에만 남아 있어, 로딩 → 렌더 전환 시 모서리 모양이 튀었습니다.WishCardSkeleton)WishCard) — 위시리스트·위시에서 가져오기 양쪽에 적용4. 단일 입력 다이얼로그에서 엔터로 제출
입력 후 버튼을 눌러야만 진행되던 곳들을 엔터로도 제출되게 했습니다.
상품 파싱 실패 시 입력 폼(상품명 + 가격)은 의도적으로 제외했습니다. 저장 조건에 이미지 선택이 포함돼 키보드만으로 완결되지 않고, 다중 필드에서는 엔터를 "다음 칸 이동"으로 기대하는 경우가 많으며, 바로 옆에 삭제 버튼이 있어서입니다. 단일 입력은 엔터 제출 / 다중 필드는 클릭으로 기준을 잡았습니다.
Summary by CodeRabbit
개선 사항
버그 수정