Skip to content

Conversation

@heeyozip
Copy link
Collaborator

@heeyozip heeyozip commented Jan 23, 2025

✏️ 한 줄 설명

이 PR의 주요 변경 사항이나 구현된 내용을 간략히 설명해 주세요.

기존 useZoomSpace에 strategy 패턴을 활용해 줌과 이동 로직을 분리하였습니다.

✅ 작업 내용

  • 기존 useZoomSpace에서 휠 이벤트가 발생했을 때의 로직과 위치 이동 로직을 각각의 전략으로 분리

📌 리뷰 진행 시 참고 사항

리뷰 코멘트 작성 시 특정 사실에 대해 짚는 것이 아니라 코드에 대한 의견을 제안할 경우, 강도를 함께 제시해주세요! (1점: 가볍게 참고해봐도 좋을듯 ↔ 5점: 꼭 바꾸는 게 좋을 것 같음!)

  • 포트 충돌 문제로 아직 제대로 테스트해보지 못한 상황입니다. 충돌 해결 후 테스트 완료되면 머지할 예정이니, 정상적으로 동작하는지만 확인 부탁드립니다!

@heeyozip heeyozip self-assigned this Jan 23, 2025
@heeyozip heeyozip requested a review from parkblo January 23, 2025 04:04
@parkblo
Copy link
Collaborator

parkblo commented Jan 23, 2025

serghaaz

마우스 휠 이벤트로 줌인, 줌아웃, 휠을 통한 이동, 홀드를 통한 이동
모두 정상 작동 확인했습니다! 👍 고생하셨습니다

@heeyozip heeyozip merged commit b384d6b into dev Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants