Conversation
Walkthrough이 변경사항은 이벤트 등록 단계 네비게이션을 인덱스 기반에서 단계명 기반으로 리팩토링하고, 여러 UI 컴포넌트의 반응형 텍스트 크기 및 차트 크기를 조정합니다. 또한 메뉴 버튼 데이터에 backPath 속성을 추가하고, 관련 네비게이션 로직을 업데이트하였습니다. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant EventFunnel
participant StepComponent
participant Navigation
User->>StepComponent: 단계 내에서 '다음' 또는 '이전' 클릭
StepComponent->>EventFunnel: goTo(단계명)
EventFunnel->>Navigation: onNext(단계 인덱스)
Note right of EventFunnel: 마지막 단계에서 handleCreateEvent 호출
EventFunnel->>Navigation: 이벤트 생성 성공 시 단계 이동 또는 에러 처리
Possibly related issues
Suggested labels
Suggested reviewers
Poem
✨ Finishing Touches
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
src/features/dashboard/ui/MultiplePieCharts.tsx (1)
33-36: 고정 높이·반경 하드코딩 대신 반응형 값 고려 필요350 px/90 px로 하드코딩하면 모바일(세로) 화면에서 차트가 잘리거나 스크롤이 과도하게 발생할 수 있습니다.
height·outerRadius를min(containerWidth, 350)또는useWindowSize훅으로 계산해 주면 레이아웃 깨짐을 방지할 수 있습니다.-<ResponsiveContainer width="100%" height={350}> +<ResponsiveContainer width="100%" height={Math.min(350, window.innerWidth * 0.8)}> ... -<Pie ... outerRadius={90} ... +<Pie ... outerRadius={Math.min(90, (window.innerWidth * 0.8) / 4)} ...또는
aspect={1}를 활용해 비율 기반으로 조정하는 방법도 있습니다.src/pages/menu/ui/MenuPage.tsx (1)
11-14: state 값 하드코딩 개선 검토네비게이션 로직이 올바르게 구현되었으나, 13번째 줄에서
backPath: '/menu'가 하드코딩되어 있습니다. 전달받은backPath파라미터를 사용하는 것이 더 유연할 것 같습니다.다음과 같이 수정하는 것을 고려해보세요:
- state: { backPath: '/menu' }, + state: { backPath },
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (6)
src/features/dashboard/ui/MultiplePieCharts.tsx(1 hunks)src/features/event/ui/EventFunnel.tsx(8 hunks)src/pages/menu/ui/MenuPage.tsx(2 hunks)src/shared/types/menuType.ts(2 hunks)src/widgets/dashboard/ui/response/ResponseModal.tsx(1 hunks)src/widgets/dashboard/ui/ticket/SelectTicketInfo.tsx(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: storybook
🔇 Additional comments (14)
src/widgets/dashboard/ui/response/ResponseModal.tsx (1)
34-34: 반응형 텍스트 클래스 추가 👍
sm:text-sm추가로 작은 화면 가독성이 개선되었습니다. 별다른 문제 없습니다.src/widgets/dashboard/ui/ticket/SelectTicketInfo.tsx (1)
15-16:text-14커스텀 유틸리티 클래스 존재 여부 확인 필요Tailwind 기본 스케일에는
text-14가 없습니다.
프로젝트에서 확장한 유틸리티라면 문제 없지만, 누락 시 빌드 타임에 경고가 발생하고 기본값(1rem)으로 폴백됩니다.text-base+tracking-…등으로 대체하거나tailwind.config.js의theme.fontSize에 14 px을 등록했는지 확인해주세요.src/shared/types/menuType.ts (2)
25-25: 타입 정의가 올바르게 추가되었습니다선택적
backPath속성이 적절히 정의되었습니다. 이는 이전 페이지로 돌아가는 네비게이션 기능을 지원하는 데 필요한 타입입니다.
34-34: 이벤트 호스팅 버튼에 backPath 설정이 적절합니다이벤트 생성 플로우에서 메뉴 페이지로 돌아가는 경로가 명확하게 설정되었습니다. 이는 사용자 경험을 개선하는 좋은 변경사항입니다.
src/pages/menu/ui/MenuPage.tsx (3)
3-3: 필요한 타입 임포트가 올바르게 추가되었습니다네비게이션 옵션을 지원하기 위해 필요한 타입들이 적절히 임포트되었습니다.
8-8: 함수 시그니처가 올바르게 업데이트되었습니다
backPath파라미터가 추가되어 뒤로 가기 네비게이션을 지원할 수 있게 되었습니다. 타입 안전성도 확보되었습니다.
34-34: onClick 핸들러가 올바르게 업데이트되었습니다새로운
backPath속성이 적절히 전달되고 있습니다. 이는 menuType.ts의 변경사항과 일관성이 있습니다.src/features/event/ui/EventFunnel.tsx (7)
10-10: 필요한 훅과 상태 관리가 올바르게 추가되었습니다
useLocation훅과useState가 추가되어 동적 backPath 관리가 가능해졌습니다. 네비게이션 상태에서 backPath를 가져오는 로직이 적절합니다.Also applies to: 16-16, 23-24
18-18: 컴포넌트 인터페이스 단순화
onPrev프로퍼티가 제거되어 컴포넌트 인터페이스가 단순해졌습니다. 이는 새로운 단계명 기반 네비게이션 시스템과 일관성이 있습니다.
26-41: 단계명 기반 네비게이션 시스템이 훌륭합니다
stepOrder배열과goTo함수를 통한 단계명 기반 네비게이션은 매우 우수한 리팩토링입니다. 코드 가독성과 유지보수성이 크게 향상되었습니다.
43-55: 이벤트 생성 로직 분리가 적절합니다
handleCreateEvent함수를 별도로 분리하여 단일 책임 원칙을 잘 따르고 있습니다. 성공/실패 시 네비게이션 로직도 명확합니다.
67-67: 호스트 생성 성공 핸들링이 개선되었습니다
goTo(StepNames.HostSelection)을 사용하여 명확한 단계 이동을 구현했습니다. 이는 새로운 네비게이션 시스템과 일관성이 있습니다.
81-82: 모든 단계별 네비게이션이 일관되게 업데이트되었습니다각 단계의
onNext와onPrev핸들러가goTo함수를 사용하여 명확한 단계명으로 네비게이션하도록 업데이트되었습니다. 특히 마지막 단계에서handleCreateEvent()를 호출하는 로직이 적절합니다.Also applies to: 92-92, 101-102, 111-112, 120-121, 130-131, 140-141, 148-149
82-82: 동적 backPath 사용이 훌륭합니다메뉴 페이지에서 전달받은
backPath상태를 활용하여 초기 단계의 뒤로 가기 네비게이션을 구현한 것이 매우 좋습니다. 이는 사용자 경험을 크게 개선합니다.
2 participants
Summary by CodeRabbit
신규 기능
버그 수정
스타일
리팩터링