-
Notifications
You must be signed in to change notification settings - Fork 13
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
disable max zoom? #4
Comments
I've made a few commit to add a new option, and to document it. Now there is a stretch option, default value is 1, but you can put what float you want greater than 1 (2 allow to zoom up to 2x ...). In the code, it wasn't minPercent the pb, but the "1" right next to it ;) And about the click controls only zoom 1px per click... since the commit i couldn't reproduce the bug! Hope this help. I'm waiting your response to close the issue. |
Hi Adrien, I made a fiddle to show you what I mean. btw, there is a typo *stretch Daniel |
Hi, Oops, That's what happend when I code after 2 am when my son wakes me up! The latest commit should fix this, thanks for the fiddle and typo ;) Adrien 2013/10/13 niestudio notifications@github.com
|
you can test it, i've updated the js fiddle: http://jsfiddle.net/szZGe/3/ |
brilliant! |
merci, Adrien 2013/10/13 niestudio notifications@github.com
|
Thanks for this plugin, I have been using it for a while, and even adapted it to use a jquery ui slider instead of the buttons.
An "issue" I'm facing now is if I have { width: 200, height:200 } and the user uploads an image that is <= 200 x 200 the zoom controls do nothing because minPercent is already >= 1.
The other scenario that makes the behavior strange is if the user uploads an image @ 210x210. the click controls only zoom 1px per click (assuming { step:10 } ).
Even though images would get pixelated, I think I'd like to allow them to zoom past minPercent.
Any thoughts on a tweak to allow this?
thanks
The text was updated successfully, but these errors were encountered: