[Feat/#70] 사물함 구역 선택 화면 - #71
Open
sangrae2325 wants to merge 8 commits into
Open
sangrae2325 wants to merge 8 commits into
sangrae2325 wants to merge 8 commits into
Conversation
혼잡도 뱃지는 WDS ContentBadge를 쓴다 — 여유/보통/마감 임박은 accent, 마감은 neutral이다. 마감 배경색 토큰(cool-neutral-97)을 새로 추가했다.
평면도는 3열 고정 배치다. 호실·화장실·계단 위치는 실제 건물 구조라 레이아웃이 갖고, 구역과 잔여 수량만 데이터에서 온다.
확인용 임시 화면이던 걸 실제 진입 화면으로 바꿨다. 들어오면 시트를 띄우고 확인하면 구역 선택으로 보낸다. 신청 기간 여부는 API 전까지 목 상수로 둔다.
없는 토큰(border-line-solid-normal)을 써서 Tailwind가 클래스를 만들지 못했고 테두리가 currentColor로 떨어져 있었다. Line/Solid/Neutral(#eaebec)과 값이 다른 별개 토큰이다.
- 용어를 zone에서 section으로 통일 (라우트 /lockers/apply/sections 포함) - Figma 치수를 flex 비율로 옮겨 뷰포트 높이·폭을 채우도록 변경. 데스크톱 컬럼(480px)과 큰 폰에서 아래가 비던 문제를 없앤다 - 선택 시에만 테두리가 생겨 콘텐츠가 1px 밀리던 것을 투명 테두리로 고정 - 새로고침 아이콘을 Figma의 Icon/Normal/Reset으로 맞추고 동작 연결 - 미선택일 때도 버튼 문구를 신청하기로 두고 잠그기만 한다
|
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 |
sangrae2325
marked this pull request as ready for review
September 26, 2026 15:16
github-actions
Bot
requested review from
jjunh33,
leegain1,
tnals0924 and
xeoxxn
September 26, 2026 15:16
This was referenced Sep 27, 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.
#️⃣연관된 이슈
🎯 해결하려는 문제가 무엇인가요?
사물함 신청에서 구역을 고르는 화면이 없다. Figma
사물함 구역 선택 전(1737:218452) →선택완료(1737:218489)를 옮긴다.#68에서 만든 유의사항 시트의
신청하기가 갈 곳도 여기다.❓ 왜 해결해야 하나요?
유의사항 시트만 있고 그다음이 비어 있어서 신청 흐름이 끊긴다. 구역마다 잔여 수량과 혼잡도가 달라서, 평면도 위에서 어디가 남았는지 보고 고를 수 있어야 한다.
실제화면
⭐ 어떻게 해결했나요?
평면도
↑ 정릉시장 방향은 실제 건물 구조라 데이터가 아니라 레이아웃이 갖는다구역 카드 5상태
Background/Normal/AlternativeContentBadge color="accent"+ green / orange / redCool Neutral/97color="neutral"(배경Fill/Normal), 선택 불가Blue/95+Primary/Normal1px뷰포트에 맞춰 비례 배치 — Figma는 375×812 한 벌만 있는데
ScreenLayout은 폰이든 데스크톱 컬럼(480px)이든 높이를 뷰포트로 꽉 채운다. 치수를 고정하면 큰 화면에서 아래가 크게 빈다. Figma 값을 그대로 flex 비율로 옮겨(열 124:67:124, 왼쪽 열 186:74:186:134) 남는 공간을 비례 배분한다. 375×812에서는 Figma와 같은 치수가 나온다.#68 시트와 연결 — 확인용 임시 화면이던
LockersApplyScreen을 신청 흐름의 첫 화면으로 바꿨다. 들어오면 시트를 띄우고신청하기를 누르면 구역 선택으로 간다.🧩 이 PR의 한계 & 트레이드오프
선택하기이후 화면이 없다. Figma에서 찾지 못해 지금은 아무 동작도 하지 않는다128px까지 커지는데 글자는 1112px 그대로라 안이 허전해진다. 상한을 둘지는 정하지 못했다window.location.reload()다. 잔여 수량 API가 붙으면 그 조회만 다시 하도록 바꿔야 한다6/6은 여유인데24/60은 보통,24/40은 마감 임박이라 비율과 맞물리지 않는다마감으로 뒀다. Figma 선택 전 프레임은여유 24/100, 선택 후 프레임은마감 0/100이다. 선택과 무관한 변화라 마감 상태를 보여주려는 의도로 읽었다⛓️ 기존 기능에 미치는 영향
feat/#68-lockers-notice-sheet라 그 브랜치의 커밋이 함께 보인다index.css에 토큰 2개 추가 —cool-neutral-97(마감 배경),line-solid-normal(평면도 점선). 둘 다 WDS 변수 별칭이다/lockers/apply가 확인용 임시 라우트에서 실제 진입 화면이 됐다. 진입점(홈 퀵링크·공지 CTA)은 아직 없어서 URL로만 들어갈 수 있다🔀 Edge Case & 실패 시나리오
신청하기그대로다(Figma는구역을 선택해 주세요로 바뀌지만 문구를 고정하기로 정했다)📋 검토한 대안과 선택 이유
ScreenLayout이 "높이는 뷰포트를 꽉 채운다"를 전제로 만들어져 있어 비례 배치로 갔다sx={{ height: "56px" }}로 버튼 높이 맞추기 → 문구가 길어질 때 잘려서 문서화된paddingBlock쪽을 썼다💬 리뷰 포인트
[c]평면도를 flex 비율로 옮긴 방식 — Figma에 375×812 외 기준이 없어서 이 PR에서 처음 정하는 규칙이다. 기존 화면은 전부 목록형(스크롤)이거나 짧은 내용형이라 선례가 없었다[c]구역 용어를section으로 통일했다(/lockers/apply/sections,LockersSectionCard).terminology.md에는 넣지 않았다 — 사전이 전부 최상위 도메인이라 그 하위 개념을 넣는 게 맞는지 애매했다[c]임시 화면이던LockersApplyScreen을 실제 진입 화면으로 승격시킨 것[a]긴 창에서 카드 높이 상한을 둘지[a]목데이터 C 구역을마감으로 둔 것🎨 디자인 확인 요청
px-16이고 나머지는px-7이다(내용이 가운데 정렬이라 화면상 차이는 없다).1737:218476만 레이어 이름이Locker Button이고 나머지는Zone Button인데 구조·스타일은 같다선택하기이후 화면이 있는지✅ 검증
tsc -b·biome check·vite build통과getComputedStyle실측으로 Figma 대조 — Floor Map 335×594, 열 간격 10, 왼쪽 열 gap 4 / 오른쪽 6, 카드 124×74·radius 8·px 7·gap 4, 마감 배경#EAEBEC, 뱃지 32×20·px6 py3·11px·green 8%, 점선#E1E2E4모두 일치