-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
feat: zoom feature [#35, #95, #403] #520
Conversation
테스트해볼 수 있는 데모가 있으면 좋겠네요. |
@lja1018 저도 같은생각이에요. 브랜치 checkout 하고 나서 테스트 하려면 어떻게 해야 하는지 가이드 해주거나요. |
@dongsik-yoo @junghwan-park |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
리뷰 완료합니다. 줌이 생각보다 섬세하게 구현해야 하는데 잘하셨습니다. 수고하셨어요.
- 스크롤바를 마우스로 움직이거나 마우스 휠로 스크롤이 이동되는 기능도 있으면 좋겠네요.
AS-IS
TO-BE마지막에 누른 모드만 반영되어야함. |
@lja1018 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
코드 리뷰 완료입니다!
Zoom 기능 기다린 분들이 많을텐데 얼른 버튼 디자인도 나와서 적용되면 좋겠네요 ㅎㅎ
사용성 측면에서는 조금 더 고민이 필요할 것 같습니다.
저는 Zoom을 할 Area를 정해서 zoom 하는게 맞을지 잘 모르겠네요.
Zoom In
단추를 누르면 전체가 확대된다고 생각했는데 그게 아니라서 약간 어색했어요
지금은 Mouse Scroll 지원도 빠져있어서...
이미지를 크게 확대 시킨 뒤에, 작은 화면을 Hand 툴만으로 조금씩 이동시키기엔 불편하기도 하고요.
waiting for this feature.... |
Pan and Zoom using mouse gestures: No changes in sources, i use a separete class: Only need: call new IEditorPanZoom(imageEditor).enable(); I haven't tested it on iPhone yet ...
|
Please check if the PR fulfills these requirements
fix #xxx[,#xxx]
, where "xxx" is the issue number)Description
작업 내역
기능
확대 기능
핸드툴
Thank you for your contribution to TOAST UI product. 🎉 😘 ✨