{"action":"created","issue":{"url":"https://api.github.com/repos/requtize/cropimg/issues/7","labels_url":"https://api.github.com/repos/requtize/cropimg/issues/7/labels{/name}","comments_url":"https://api.github.com/repos/requtize/cropimg/issues/7/comments","events_url":"https://api.github.com/repos/requtize/cropimg/issues/7/events","html_url":"https://github.com/requtize/cropimg/issues/7","id":77270819,"number":7,"title":"Zoom interval infinite loop","user":{"login":"Noppey","id":12481823,"avatar_url":"https://avatars.githubusercontent.com/u/12481823?v=3","gravatar_id":"","url":"https://api.github.com/users/Noppey","html_url":"https://github.com/Noppey","followers_url":"https://api.github.com/users/Noppey/followers","following_url":"https://api.github.com/users/Noppey/following{/other_user}","gists_url":"https://api.github.com/users/Noppey/gists{/gist_id}","starred_url":"https://api.github.com/users/Noppey/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Noppey/subscriptions","organizations_url":"https://api.github.com/users/Noppey/orgs","repos_url":"https://api.github.com/users/Noppey/repos","events_url":"https://api.github.com/users/Noppey/events{/privacy}","received_events_url":"https://api.github.com/users/Noppey/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-05-17T10:01:24Z","updated_at":"2015-05-19T22:34:24Z","closed_at":null,"body":"Hi,\r\n\r\nIt seems like when you click-and-hold the zoom-in button, and drag this button somewhat to the right, the interval that triggers the zoom doesn't get cleared. On mobile phones, this is also troubling, since you can't click-and-hold, so you will automatically will trigger this behavior when you try to zoom.\r\n\r\nI solved this by changing the drawZoomingButtons to div's, instead of a-elements. This isn't a complete solution for mobile, since you can still not click-and-hold, but at least you won't end up with a non-clearable interval.\r\n\r\nPerhaps you know a better solution that could fix mobile cropping completely.\r\n\r\nCompliments on your project by the way, I tried a few croppers, but this was the first one that actually worked out of the box for me."},"comment":{"url":"https://api.github.com/repos/requtize/cropimg/issues/comments/103685891","html_url":"https://github.com/requtize/cropimg/issues/7#issuecomment-103685891","issue_url":"https://api.github.com/repos/requtize/cropimg/issues/7","id":103685891,"user":{"login":"requtize","id":7622472,"avatar_url":"https://avatars.githubusercontent.com/u/7622472?v=3","gravatar_id":"","url":"https://api.github.com/users/requtize","html_url":"https://github.com/requtize","followers_url":"https://api.github.com/users/requtize/followers","following_url":"https://api.github.com/users/requtize/following{/other_user}","gists_url":"https://api.github.com/users/requtize/gists{/gist_id}","starred_url":"https://api.github.com/users/requtize/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/requtize/subscriptions","organizations_url":"https://api.github.com/users/requtize/orgs","repos_url":"https://api.github.com/users/requtize/repos","events_url":"https://api.github.com/users/requtize/events{/privacy}","received_events_url":"https://api.github.com/users/requtize/received_events","type":"User","site_admin":false},"created_at":"2015-05-19T22:34:24Z","updated_at":"2015-05-19T22:34:24Z","body":"I've test plugin behavior on situation you describe. There is a problem, but, i think there is really small percentage of people, who drag zoom-in button instead of clicking it. Buttons are designed for clicking, not for dragging. Zoom buttons have interval for zooming while user hold-up clicking (mousedown), and until mouseup event for body wasn't fires, zooming will be working. That's why, if you drag zoom button, zooming work all the time - because mouseup event wasn't trigger.\r\n\r\nOn mobile devices, you have right, maybe DIV instead of A element, will be better solution for buttons. I will think of this in bugfixes."}}