Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
github-actions
Bot
requested review from
jjunh33,
sangrae2325,
tnals0924 and
xeoxxn
September 27, 2026 12:38
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의 A-1구역(
2159:110753), A-2구역(2159:109533), A-1구역 실제사진(2159:110174), 사물함 선택 시(2159:113197)를 옮긴다.❓ 왜 해결해야 하나요?
#71의 구역 선택 화면에서
신청하기를 눌러도 갈 곳이 없어서 신청 흐름이 끊겨 있다. 구역마다 칸 배치가 실제 공간 구조대로 달라서, 도면처럼 보고 칸을 골라야 한다.⭐ 어떻게 해결했나요?
화면 흐름
신청하기→/lockers/apply/sections/:sectionId→ 칸 선택준비 중화면을 보여준다칸 배치
LockersA1LockerMap,LockersA2LockerMap)를 둔다. 창문·벽면·옆 구역·호실은 실제 공간 구조라 [Feat/#70] 사물함 구역 선택 화면 #71 평면도처럼 레이아웃이 갖고, 칸 번호·선택 가능 여부만 데이터다하단 선택 영역 —
LockersSelectedLockerBar로 분리. 선택 전엔선택한 사물함이 없어요.+ 잠긴사물함을 선택해 주세요, 선택 후엔A-25(Primary SemiBold) +사물함 신청하기실제 사진 모달 — A-1 헤더의 사진 아이콘으로 연다. 사진 위에 칸 번호 핀을 Figma 좌표 그대로 얹었다(위쪽 칸부터 Orange/50 → 70 → 90). 딤 클릭·Esc로 닫힌다
WDS —
ScreenHeader+TopNavigationButton,IconImage/IconReset/IconClose,IconButton,ActionArea/ActionAreaButton,Typography. 칸·칸 묶음·미니맵·범례·모달은 Stream 로컬이다(wds-component-usage.md에 정리).🧩 이 PR의 한계 & 트레이드오프
사물함 신청하기이후 동작이 없다. 신청 API가 없어서 버튼만 열어뒀다(onSubmit만 채우면 된다)window.location.reload()다. [Feat/#70] 사물함 구역 선택 화면 #71 구역 선택 화면과 같은 방식이다zoom: 0.31로 줄여 그려서 칸·글자 크기가 Figma 미니맵(칸 8px·글자 4px)과 조금 다르다. 대신 배치가 바뀌어도 미니맵을 따로 고칠 필요가 없고, 테두리를 스크롤 비율에 정확히 맞출 수 있다⛓️ 기존 기능에 미치는 영향
LockersSectionSelectScreen의신청하기가 이제 칸 선택 화면으로 이동한다(원래는 아무 동작 없음)index.css에 토큰 5개 추가 —orange-95/90/70/50,line-solid-alternative. 전부 WDS 변수 별칭이다🔀 Edge Case & 실패 시나리오
/lockers/apply/sections/B-1등):준비 중화면replace이동opacity: 0으로 숨겨 자리만 두었다inert로 막아 Tab·스크린리더에 두 번 잡히지 않게 했다key={sectionId}로 새로 그린다📋 검토한 대안과 선택 이유
💬 리뷰 포인트
[c]하단 사물함 번호 표기 — Figma가 B-1구역 25번을B-25로 적어서, 구역 번호 없이 동 이름 + 칸 번호로 표기했다(A-25). 같은 동 안에서 칸 번호가 겹치지 않는다고 읽었다[c]미니맵을zoom으로 축소한 방식[a]A-1Shelf Label— Figma 본문 프레임에서 높이가 1px로 깨져 있어서, 미니맵·A-2처럼 칸 묶음 높이(122px)에 맞췄다[a]실제사진 모달 — Figma는 top 229px이지만 화면 높이가 기기마다 달라 세로 가운데에 뒀다✅ 검증
tsc -b·biome check·vite build통과