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 on touch devices at center of touch contacts instead of center of canvas. #788

Closed
nmengers opened this issue Dec 7, 2015 · 6 comments
Labels

Comments

@nmengers
Copy link

@nmengers nmengers commented Dec 7, 2015

Hello,

I'm not sure if this question has been asked before but I could not really find it using the search function.

When I pinch to zoom in on an OpenSeaDragon canvas the zooming always seems to happen on the center of the canvas instead of the centerpoint between the touch contacts as I'm used to when I'm using native zooming on my touch device.

Would it not be a nice feature to be able to also zoom in on the place between the touchcontacts by for example setting an option?

@iangilman

This comment has been minimized.

Copy link
Member

@iangilman iangilman commented Dec 7, 2015

Good point! This would be a great enhancement. Would you be interested in looking into implementing it? @msalsbery might have some thoughts to add.

@msalsbery

This comment has been minimized.

Copy link
Member

@msalsbery msalsbery commented Dec 7, 2015

It used to work that way. I noticed it was broken a while back and it's on my todo list

@nmengers

This comment has been minimized.

Copy link
Author

@nmengers nmengers commented Dec 8, 2015

I could give it a try. Where do you suggest I start searching for the problem @msalsbery ? Could you give me some tips. Or do you prefer to solve this yourself?

@illtud

This comment has been minimized.

Copy link

@illtud illtud commented Feb 1, 2017

This is being addressed in #800 ?

@iangilman

This comment has been minimized.

Copy link
Member

@iangilman iangilman commented Feb 1, 2017

Yes, I believe fixing #800 should fix this.

@iangilman

This comment has been minimized.

Copy link
Member

@iangilman iangilman commented May 11, 2017

#800 has been fixed by #1184, so I believe that fixes this as well.

@iangilman iangilman closed this May 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.