[Feat/#26] 행사 목록 화면 구현 - #27
Open
leegain1 wants to merge 7 commits into
Open
Conversation
…rops) Figma의 Top Navigation 패턴(타이틀 정렬, leading/trailing 조합, 게시판류 토글형 2단 타이틀)을 props로 고르게 만들었다. display variant에서는 leading을 타입으로 막았다 — WDS 쪽 스타일이 display일 때 leading/trailing 포지셔닝을 안 줘서 레이아웃이 깨진다.
github-actions
Bot
requested review from
jjunh33,
sangrae2325,
tnals0924 and
xeoxxn
September 12, 2026 07:53
xeoxxn
force-pushed
the
refactor/#23-unified-screen-header
branch
from
September 14, 2026 11:47
506375f to
bfa23b1
Compare
5 tasks
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.
#️⃣연관된 이슈
🎯 해결하려는 문제가 무엇인가요?
Bottom Nav "행사" 탭에 대응하는 목록 화면이 없어서 탭을 눌러도 갈 곳이 없었다. Figma "행사"(nodeId
1243:70854)를/events라우트로 구현한다.❓ 왜 해결해야 하나요?
⭐ 어떻게 해결했나요?
FilterChipGroup공용화: 행사 상태 필터 칩을get_design_context로 열어보니 원본이1016:55355— 빌릴게 카테고리 필터와 동일한 Stream 로컬 칩이었다. 두 화면이 같은 칩을 쓰는 게 확인돼components/ui/로 올리고BililgeCategoryFilter는 상수 + 호출만 남겼다(52줄 → 31줄).ScreenHeader에toolbar슬롯 추가: FigmaTop Navigation/Resource/Contents(h=88)는Navigation(056) +88, 세그먼트 토글) 구조이고, WDSTool(56TopNavigation에 이미 대응하는toolbarprop("Area attached below the navigation")이 있다. 그래서 행사/신청내역 토글을 화면 본문이 아니라 헤더로 넘겼다.EventCard: 모집중/모집예정/모집종료 3종. 뱃지는 WDSContentBadge, CTA는 WDSButton(size="small").뷰
🧩 이 PR의 한계 & 트레이드오프
#d9d9d9이미지 슬롯이라 단색 placeholder로 뒀다. 전용 토큰(--color-thumbnail-placeholder)에 "실 API 붙으면 사라질 자리"라고 주석을 남겼다.toolbar로 옮겨서 두 화면 방식이 갈렸다 — 인접 코드 임의 수정을 피하려고 후속으로 미뤘다(AGENTS.md 작업 원칙).⛓️ 기존 기능에 미치는 영향
FilterChipGroup을 쓰도록 바뀌었다. 렌더링 결과가 동일한지 실측으로 확인했다.ScreenLayout의 Bottom Nav 행사 탭 경로를/event→/events로 고쳤다(용어 사전 코드 용어events).ScreenHeader의toolbar는 선택 prop이라 기존 화면(홈·빌릴게)에 영향 없다.🔀 Edge Case & 실패 시나리오
/board)은 여전히 라우트가 없어서 누르면 빈 화면이 된다 — 이 PR 범위 밖이지만 [Feat/#21] 빌릴게 페이지 목록 화면 구현 #22 리뷰에 남겨둔 사항이다.📋 검토한 대안과 선택 이유
toolbar슬롯을 이미 갖고 있어서 우리가 새 패턴을 만드는 게 아니고, 본문에 두면 헤더 고정 영역 밖이라 스크롤 경계가 화면마다 달라진다.component-convention.md§3의 "새 헤더 패턴이 필요하면 ScreenHeader부터 확장한다"와도 맞다.EventStatusBadge래퍼 컴포넌트: 만들려다 뺐다.ContentBadge의color="accent"|"neutral"분기가 Figma의 상태별 배경 처리(accent는 8% 오퍼시티, neutral은fill.normal)와 그대로 1:1이라, 상태 → 토큰 매핑 상수만으로 충분했다.sx색 보정: 불필요했다. WDSButton의&[aria-disabled='true']색(interaction.disable+label.assistive)이 Figma 비활성 스펙과 정확히 같아서disabledprop만 줬다. (빌릴게 카드 버튼은sx보정이 필요했던 것과 대조됨)💬 리뷰 포인트
[r]Figma 대비 남은 차이 2건이 있는데 둘 다 이 화면이 원인이 아닙니다. ① WDSTopNavigation의 Navigation 행이 64px인데 Figma는 56px(빌릴게 헤더도 실측 64px) ② 필터 칩 높이가 35px인데 Figma는 32px(빌릴게에서 그대로 가져온 값). 누적 11px 밀립니다.ScreenHeader/FilterChipGroup차원의 결정이 필요해 이번 범위에서 뺐는데, 별도 이슈로 잡을지 봐주세요.[c]빌릴게 토글도toolbar로 옮겨서 두 화면 방식을 맞출지 판단 부탁드립니다.[a]BottomNavValue는event(단수)인데 라우트는/events입니다. 탭 식별자라 그대로 뒀는데 통일하는 게 나을지요.