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

zoom/rotate based on center of screen instead of center of image #202

Closed
codemasterover9000 opened this issue Sep 6, 2018 · 3 comments
Closed

Comments

@codemasterover9000
Copy link

Hi,

Image manipulation tools (zoom/rotate) all operate based on the center of the image. So when for example the image is very tall and zoomed in. The center of the image is way outside the screen. Rotating the image will make it disappear.

Making the operations work based on the center of the screen fixes these problems and feels more natural to the end user (IMO).

Thanks

@achilleus68
Copy link

Same goes for the '1:1' button:
when zoomed in on a specific part of the image (say the left top corner) and pressing the '1:1' button, the images disappears from the screen. Pressing the 'reset' button re-positions the image.

IMHO this is a bug, not an enhancement.

@fengyuanchen
Copy link
Owner

  1. When calling the zoom or rotate methods, the zoom / rotate center is the center of the canvas (image wrapper).
  2. When zoom by wheel or touch, the zoom center is the position of the pointer.

@achilleus68
Copy link

But it should be possible the center the image on screen when pressing the '1:1' button.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants