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

Fix issue where it does not capture signature on desktop browsers #203

Merged
merged 1 commit into from
Mar 27, 2019

Conversation

hassaanalansary
Copy link
Contributor

  • Issue: when trying to sign in the box, it just add dots, as opposed to a continuous line.
    The issue happens on [Chrome 73.0.3683.86 (Official Build) (64-bit), Opera 58.0.3135.107, Edge], Firefox 66.0.1 (64-bit) works very well.

  • This fix fixes the issue for [Chrome, Opera], however it doesn't fix it for Edge. and still works for Firefox. also tested on [Android Chrome, Android Opera Touch]

  • Explanation: There is a problem with this.ontouchstart = function (e)doesn't appear to function well, however eddEventListener('touchstart', function(e) does work

Issue: when trying to sign in the box, it just add dots, as opposed to a continuous line. 
The issue happens on [Chrome 73.0.3683.86 (Official Build) (64-bit), Opera 58.0.3135.107, Edge], Firefox 66.0.1 (64-bit) works very well.
This fix fixes the issue for [Chrome, Opera], however it doesn't fix it for Edge. and still works for Firefox. also tested on [Android Chrome, Android Opera Touch]
Explanation: There is a problem with ```this.ontouchstart = function (e) ``` doesn't appear to function well, however ```eddEventListener('touchstart', function(e)``` does work
@mockdeep mockdeep merged commit c65abed into thread-pond:master Mar 27, 2019
@mockdeep
Copy link
Collaborator

Excellent, thanks for your contribution @hassaanalansary!

@hassaanalansary
Copy link
Contributor Author

thank you.
Disclaimer: after further testing, I found that with this fix when you start using one input method, it stops the other.
meaning that if you start by touching, you can't use your mouse to sign, and vice versa

@mockdeep
Copy link
Collaborator

@hassaanalansary it looks like we have a number of places where we set this.ontouchstart = null. Maybe we should change those to remove the event listener instead?

@hassaanalansary
Copy link
Contributor Author

sorry for the delay I was busy these last 2 days.
You are right, It appears in 2 places, after hours of testing, changing them will fix a minor problem.
I will create a new pull request and will explain what it fixes there.

@curlyroger
Copy link

I'm having this issue on a large kiosk touchscreen where the signature pad is inside an iframe. Inside the frame, I can only get dots and horizontal lines. Outside of the iframe, the signature works fine. But inside the iframe, the behavior appears. This happened recently with a Chrome update and also happens in Firefox too. Any thoughts? I am using the latest version which has this fix in it but it doesn't help.

@hassaanalansary
Copy link
Contributor Author

@curlyroger does this happen on mobile devices also?

Try to use the older version before my fix.
However i don't have any experience with iframes whatsoever.

@curlyroger
Copy link

curlyroger commented Nov 20, 2019 via email

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

Successfully merging this pull request may close these issues.

None yet

3 participants