Repository navigation
Conversation
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
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 |
github-actions
Bot
requested review from
jjunh33,
leegain1,
sangrae2325 and
tnals0924
October 3, 2026 17:03
Figma 모달 설명은 줄바꿈 위치까지 디자인에 포함돼 있는데 `\n`이 공백으로 접혀서 의도와 다른 위치에서 줄이 바뀌었다. 기존 사용처는 모두 한 줄 문구라 렌더링 변화가 없다. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
행사 신청 완료·회원 탈퇴 완료·사물함 신청 완료가 모두 같은 Figma 메인 컴포넌트(Circle Check Motion 1712:192849)의 인스턴스다. 타임라인 길이와 반복 설정은 화면마다 다르게 걸려 있지만 키프레임 값이 같아서 보이는 모션이 동일하다. json을 기능별로 복사해 두면 디자이너가 모션을 고칠 때 재export를 몇 번씩 해야 해서, Lottie와 재생 컴포넌트를 각각 common/과 components/ui/로 옮긴다. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
안내 카드 네 장이 모두 같은 Figma 컴포넌트(Delete Accoount Card 3491:151566)의 인스턴스라 WithdrawalNoticeCard 하나로 묶고 문구는 상수로 분리했다. 동의 체크박스를 켜야 탈퇴하기가 활성화되고, 누르면 확인 모달이 뜬다. WDS Button은 color가 primary/assistive뿐이라 Figma의 Status/Negative는 sx로 배경만 덮는다. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
행사 신청 완료 화면과 같은 뼈대지만 신청 정보 카드가 없고 버튼이 하나뿐이라, 표본이 둘뿐인 지금은 화면 껍데기까지 묶지 않고 체크 모션만 공유한다. 탈퇴 후에는 로그아웃 상태라 로그인/온보딩으로 나가야 하지만 그 화면이 아직 없어서 홈으로 내보낸다. 인증을 붙일 때 EXIT_PATH만 바꾸면 된다. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
사물함 화면 PR(#92)이 먼저 머지되면서 같은 작업이 양쪽에 생겼다. 중복된 것은 모두 main 쪽을 남긴다. - CompleteCheck.tsx, EventsApplicationCompleteScreen.tsx: 체크 모션을 공용으로 빼는 작업을 양쪽이 각자 했다. main은 거기서 더 나아가 결과 화면 공용 컴포넌트 (ResultScreen)까지 만들었으므로 main 쪽을 그대로 쓴다. - component-convention.md: Lottie 파일 위치 규칙을 양쪽이 같은 내용으로 고쳤다. - wds-component-usage.md, router.tsx, HomeScreen.tsx: 서로 다른 것을 더한 충돌이라 양쪽을 모두 남긴다.
main에 신청 결과 화면 공용 컴포넌트(ResultScreen)가 들어왔다. 버튼이 하나인 경우(secondaryAction 생략)와 체크 모션용 8px 간격(illustrationGap)을 이미 지원해서 탈퇴 완료 화면이 그대로 들어맞는다. 직접 조립한 마크업을 걷어낸다. 렌더링 결과는 같다 — 헤더↔체크 104px, 체크 72x72, 체크↔제목 8px, 제목↔부제 4px, 버튼 335x56으로 Figma(3524:152505)와 실측 일치를 확인했다. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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.
#️⃣연관된 이슈
구현 화면
🎯 해결하려는 문제가 무엇인가요?
Figma 설정 섹션(nodeId
1799:85920)의 화면들이 코드에 전혀 없었다. 홈 헤더에도 Figma(3147:146294)엔 설정·검색·알림 아이콘 3개가 있는데 코드엔 검색·알림 2개만 있어서 설정으로 들어갈 길 자체가 없었다.❓ 왜 해결해야 하나요?
계정 정보 확인, 알림 on/off, 전화번호 변경, 회원 탈퇴는 모두 설정 화면에서만 할 수 있는 동작이라 화면이 없으면 접근할 방법이 없다.
⭐ 어떻게 해결했나요?
커밋을 화면 단위로 나눴고, 각 커밋이 독립적으로 타입체크를 통과한다.
3013:116135) — 계정 정보 / 알림 설정 / 서비스 정보 / 로그아웃 네 장의 흰 카드 + Copyright. 카드는SettingsSection, 행은SettingsRow하나로 합쳤다(Figma의 Account Row·Navigation Row·Info Row·Action Row 넷이 높이·정렬이 같고 라벨 굵기와 색만 달라서variant/tone으로 구분).3013:116182) — 전체 알림 + 카테고리 4종. 전체 알림은 별도 state를 두지 않고 "카테고리가 전부 켜져 있는가"로 계산해서, 켜면 전부 켜지고 끄면 전부 꺼지며 카테고리를 하나라도 끄면 자동으로 꺼진다. Figma 기본 상태(빌릴게만 on → 전체 알림 off)도 이 계산과 그대로 맞는다.1799:87220) — 입력하는 대로 하이픈을 넣고(Figma 입력 중 상태1799:87230이010-1234-5형태),010+ 8자리를 다 채웠을 때만 "변경하기" 버튼이 활성화된다. 누르면 Figma처럼 완료 토스트(1917:93917)를 띄운다.3524:152473) — 안내 카드 네 장이 모두 같은 Figma 컴포넌트(Delete Accoount Card 3491:151566)의 인스턴스라WithdrawalNoticeCard하나로 묶고 문구는 상수로 분리했다. 동의 체크박스를 켜야 "탈퇴하기"가 활성화되고(#F4F4F5→ Status/Negative#FF4242), 누르면 확인 모달(3524:152517)이 뜬다.3524:152505) — 체크 모션 + 제목/부제 + "확인" 버튼.Circle Check Motion 1712:192849)의 인스턴스다.get_motion_context로 떠보니 타임라인 길이·반복 설정만 화면마다 다르고 키프레임 값(원 450ms back-out, 체크 550ms path trim)은 동일해서, Lottie json과 재생 컴포넌트를 각각assets/lottie/common/·components/ui/CompleteCheck.tsx로 옮겨 한 벌만 두도록 했다.1799:87183) — 공용ConfirmModal의tone="negative"가 Figma와 그대로 맞아 재사용했다.Switch/TextField/IconSetting매핑과, 설정 행 화살표가 WDS가 아니라는 근거를wds-component-usage.md에 추가했다./figma-check로 교차검증해서 카드 padding·radius·행 간격·타이포·스위치 크기(39×24)·버튼 높이(56px)·비활성 색(#F4F4F5)까지 실측 대조했다. 탈퇴 화면도 안내 카드(94×335)·카드 간격 8px·헤더↔제목 12px·제목↔목록 16px, 완료 화면도 헤더↔체크 104px·체크 72×72·체크↔제목 8px·제목↔부제 4px까지 맞췄다.🧩 이 PR의 한계 & 트레이드오프
SETTINGS_EXTERNAL_LINKS상수에 빈 URL만 두고 값이 있을 때만 새 탭으로 열도록 했다. 주소를 받으면 상수만 채우면 된다. 다만 앱 WebView에서는window.open이 막히거나 앱 안에 갇힐 수 있어 외부 링크 처리 방식은 앱 쪽과 맞춰야 한다.WithdrawalCompleteScreen의EXIT_PATH상수 하나만 바꾸면 되도록 빼뒀다.onConfirm에 붙인다.whitespace-nowrap). 실 데이터 들어올 때 보기로 했다.010+ 8자리로 뒀다.011/016같은 구 번호도 받아야 하면 조정이 필요하다.⛓️ 기존 기능에 미치는 영향
HomeScreen헤더에 설정 버튼이 하나 늘었다.EventsApplicationCompleteScreen이EventsCompleteCheck대신 공용CompleteCheck를 쓴다. import 경로만 바뀌었고 재생 설정(loop={false}, 모션 줄이기 처리)은 그대로다 — 72×72 SVG 렌더링과 콘솔 에러 0을 실제로 확인했다.ConfirmModal설명에whitespace-pre-line을 붙여\n을 살렸다. 기존 사용처 3곳(BililgeReturnSection·EventsApplicationScreen·FeedbacksNewScreen)은 모두 한 줄 문구라 렌더링 변화가 없다.🔀 Edge Case & 실패 시나리오
abc010가1234나5678→010-1234-5678).replace라, 뒤로가기로 탈퇴 폼이나 완료 화면에 되돌아오지 않는다.📋 검토한 대안과 선택 이유
IconChevronRight로 대체하는 것도 검토했지만,search_design_system에 같은 이름(Arrow)이 없고 글리프 여백도 WDS와 달라서 Stream 전용 아이콘으로 보고 SVG를 받아 커밋했다.1737:218058)까지 만들 때 셋을 보고 묶는 편이 낫다.💬 리뷰 포인트
[c]전체 알림을 끄면 카테고리가 전부 꺼지는 동작이 의도와 맞는지 확인 부탁드립니다(켜는 쪽만 명시적으로 정해져 있었습니다).[c]Figma에서 **탈퇴 완료 화면의 체크 모션만loop**로 걸려 있습니다(행사·사물함은once). 완료 체크가 무한 반복되는 건 다른 화면과 어긋나 프로토타입용 설정이 남은 것으로 보고 1회 재생으로 맞췄습니다 — 의도된 것이면 알려주세요.[a]뒤로가기 헤더 10줄이 이 PR의 화면들 포함 10개 파일에 중복되어 있습니다.useBackScreenHeader(title)같은 공용 훅으로 뺄 만한데, 기존 화면까지 건드리는 리팩터라 이번 PR엔 넣지 않았습니다 — 별도 이슈로 뺄까요?🤖 Generated with Claude Code