Conversation
…into feat/#22/좌석-컴포넌트-구현
Closed
3 tasks
sispo3314
approved these changes
Jul 28, 2025
Collaborator
sispo3314
left a comment
sispo3314
left a comment
There was a problem hiding this comment.
seat, seatItemProps, ModalPropsMap 등으로 타입 정리가 잘 되어있어서 전체 구조가 명확하게 보이는 것 같습니다!! 코드 가독성도 좋고 분리도 잘 되어있어서 보기 편한 것 같아요 😭 수고하셨습니다!!
| const inner = innerRef.current; | ||
| if (container && inner) { | ||
| const scrollX = (inner.scrollWidth - container.clientWidth) / 2; | ||
| container.scrollTo({ left: scrollX }); |
Collaborator
There was a problem hiding this comment.
이 부분은 처음 모달 열 때 항상 맵이 가운데에 위치하게 하는 로직인가요?? 너무 좋은 UX 처리인 것 같습니다 👍👍
Collaborator
There was a problem hiding this comment.
좌석 색상 판단 로직 이렇게 분리해두신 거 좋은 것 같아요!! 전 어떻게 나눠야 할지 헷갈려서 항상 컴포넌트에 몰아썼는데 이렇게 할 수 있군요...😭 덕분에 가독성이 좋아지는 것 같습니다👏👏
eileen4505
approved these changes
Jul 28, 2025
Collaborator
eileen4505
left a comment
eileen4505
left a comment
There was a problem hiding this comment.
전체적으로 구조와 설계가 너무 깔끔해서 리뷰하며 감탄했어요 👏
UI/UX, 타입, 상태 관리까지 모두 신경 써주신 점 인상 깊었습니다.
고생 많으셨어요! 다음 작업도 기대됩니다 🙌
soyun-git121
approved these changes
Jul 28, 2025
Collaborator
soyun-git121
left a comment
soyun-git121
left a comment
There was a problem hiding this comment.
컴포넌트안에 파일들 잘 나눠져있어서 코드들이 깔끔하고 읽기너무 좋았어요!! 고생하셨습니당
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
4 participants
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.
🔍 관련된 이슈
📝 작업 내용
📸 스크린샷
/seat 페이지는 모달 테스트를 위해 구현했고, 개별 좌석 클릭 시 /seat/review/:seatId 로 이동하도록 구현했습니당
2025-07-27.8.39.04.mov
🚨 이슈
<Modal/>이 라우터 바깥에 있어서 useNavigate를 쓰지 못하는 문제가 있는데 다음 이슈에서 헤더, 모달 컴포넌트 리팩토링을 진행한뒤 해결하도록 하겠습니다,,,
📣 리뷰 요구사항
상영관 좌석 배치도를 목데이터로 생성해서 임시로 구현했기 때문에 좌석이 실제 배치도랑은,, 좀 다르게 생겼습니당..
api 연결 후 실제 좌석배치도를 받아왔을 때 수정할 부분이 있다면 수정하겠습니다!!
✅ 체크리스트