Repository navigation
[Refactor/#120] 상태 요약 카드를 공용 컴포넌트로 통합 - #122
Open
sangrae2325 wants to merge 7 commits into
Open
sangrae2325 wants to merge 7 commits into
sangrae2325 wants to merge 7 commits into
Conversation
sangrae2325
added this pull request to stack #121
October 15, 2026 07:41
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
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 |
sangrae2325
marked this pull request as ready for review
October 15, 2026 07:42
github-actions
Bot
requested review from
jjunh33,
leegain1,
tnals0924 and
xeoxxn
October 15, 2026 07:42
This was referenced Oct 15, 2026
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.
#️⃣연관된 이슈
🎯 해결하려는 문제가 무엇인가요?
"상태 배지 + 제목 + 라벨/값 행" 요약 카드를 세 곳이 같은 마크업으로 각자 그리고 있었다(#116 리뷰에서 나온 의견).
EventsApplicationSummaryFeePaymentCard위쪽LockersAssignmentContent첫 카드❓ 왜 해결해야 하나요?
컴포넌트 컨벤션의 "두 번째 화면에서 재사용되면
components/ui/로 옮긴다" 기준을 넘었다. 지금처럼 세 벌이면 배지 크기나 행 간격이 바뀔 때 한쪽을 빠뜨리기 쉽고, 실제로 학생회비 카드만 행 간격이 달랐다.⭐ 어떻게 해결했나요?
1. 공용
StatusSummaryCard(7b65e56) —components/ui/StatusSummaryCard.tsxFEE_STATUS_BADGES등)은 각 기능constants에 그대로 두고 화면이 골라 넘긴다 — 공용 UI가 기능 상수를 import하지 않는다2. 세 곳 교체 (
7b65e56,830dd33,e99321c)EventsApplicationSummary는 "행사·신청·(취소)" 행만 만드는 파일로 남겼다3. 스켈레톤도 공용으로 (
f829a0c) — 같은 파일의StatusSummaryCardSkeleton(행 수만큼rowWidths)4. 카드 사이 간격 8px (
0b4ba53) — 디자이너가 카드 사이 간격을 8px로 통일했다(Figma4174:101198간격통일). 이 PR 화면인 학생회비 납부 건끼리, 사물함 요약↔배정 카드, 신청내역 상세 카드 사이를 12→8px로(스켈레톤 포함). 나머지 화면(행사 신청 폼·홈·열린피드백 작성내역)은 별도 이슈로5. 문서 (
13aca08,16ae84b) —wds-component-usage.md내 내역·행사 신청내역 절🧩 이 PR의 한계 & 트레이드오프
⛓️ 기존 기능에 미치는 영향
3147:147879)도 6px로 고쳐졌다. 카드 높이가 2px 줄어든다🔀 Edge Case & 실패 시나리오
📋 검토한 대안과 선택 이유
EventsApplicationSummary: 지우고 상세 화면에서 행을 만들기 vs 행만 만드는 파일로 남기기 — 학생회비·사물함도 각자 행을 만드니 같은 모양이 되도록 남겼다💬 리뷰 포인트
[a]행 간격 6px·카드 사이 8px은 디자이너 확인 완료(3147:147879,4174:101198)[a]공용 카드 이름StatusSummaryCard