Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

트랙패드 줌인줌아웃 방향 수정 #164

Merged
merged 3 commits into from
Nov 28, 2024
Merged

트랙패드 줌인줌아웃 방향 수정 #164

merged 3 commits into from
Nov 28, 2024

Conversation

CatyJazzy
Copy link
Collaborator

@CatyJazzy CatyJazzy commented Nov 27, 2024

✏️ 한 줄 설명

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

트랙패드를 통한 화면 조작 시 줌인줌아웃이 반대 방향으로 동작하던 것을 수정했어요.

✅ 작업 내용

  • deltaY 연산 방식 수정

🏷️ 관련 이슈

📸 스크린샷/영상

이번 PR에서 변경되거나 추가된 뷰가 있는 경우 이미지나 동작 영상을 첨부해 주세요.

트랙패드: 일반적으로 아는 방식으로 동작 [손가락 벌릴 때 확대] / [손가락 오므릴 때 축소]
ctrlKey + 마우스휠: 휠을 위로 굴릴 때 축소 / 아래로 굴릴 때 확대

📌 리뷰 진행 시 참고 사항

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

여러 가지 시도해봤는데, 트랙패드와 ctrlKey + 마우스휠 동작 2가지 모두를 저희가 아는 방식으로 구현하는 것에 문제가 있습니다.
일단 트랙패드의 줌인줌아웃을 보편적으로 가는 것이 더 중요하다고 생각하여 우선순위를 높게 가져갔습니다.

(macOS에서 트랙패드 제스쳐 시 ctrlKey를 자동활성화함 + 마우스휠과 트랙패드의 동작 시 deltaY 해석 방식이 달라짐)

@CatyJazzy CatyJazzy requested a review from a team as a code owner November 27, 2024 17:41
@CatyJazzy CatyJazzy requested review from hoqn and heegenie and removed request for a team November 27, 2024 17:41
@CatyJazzy CatyJazzy marked this pull request as draft November 27, 2024 17:45
@CatyJazzy CatyJazzy self-assigned this Nov 28, 2024
@CatyJazzy CatyJazzy marked this pull request as ready for review November 28, 2024 03:38
@CatyJazzy CatyJazzy merged commit cd0c601 into dev Nov 28, 2024
@CatyJazzy CatyJazzy deleted the fix/stage-zoom branch December 4, 2024 02:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant