-
Notifications
You must be signed in to change notification settings - Fork 291
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
pass arguments to getSignatureImage along to toDataURL #9
Comments
That works for me. I'll add it into the next update. |
api.getSignatureImage('image/jpeg') Is returning png format |
That should work. What browser are you using? Does it support JPGs from ? |
I am using it on an android platform 4.0.4 with phonegap, everything else seems to work, I just need to have the file as jpeg so I can add it to pdf . |
Does Phonegap have an updated enough version of Webkit? I'm not fully sure how PhoneGap works, but I was under the impression that it had its own version of Webkit. |
It uses the devices native webview kit in this case android.webkit.WebView |
Does your code work outside PhoneGap in the Android native browser? |
it is a problem with android versions 4+ converting to jpeg, for those interested this is how you get around it var imagepath = document.getElementById("signaturecanvas"); However since upgrading to 4.1.2 the signature canvas seems to only draw straight lines and misses much of the input |
Any update on this, basically it is now unusable with an android device with the latest 4.1.2 os installed. Tested your demos with the native browser and got the same result |
@clubnet So, it's unusable because you can't get JPEGs? Will PNGs not do? |
After every update, I change the declaration of getSignatureImage like so:
This means I use getSignatureImage('image/jpeg') if I want JPEGs instead of everything being PNGs. This is still broken on certain Androids, but no more broken than it was before (I think).
It would be terrific if you incorporated this in the next update, as it is completely backwards-compatible.
The text was updated successfully, but these errors were encountered: