-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Bugfix for document.ontouchstart detection #3419
Conversation
@@ -34,7 +34,7 @@ $(document).ready(function () { | |||
this.screenBlock = this.container.find('.cms_screenblock'); | |||
|
|||
// states | |||
this.click = (document.ontouchstart !== null) ? 'click.cms' : 'touchend.cms'; | |||
this.click = (document.ontouchstart == null) ? 'click.cms' : 'touchend.cms'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use ===
thanks
Updated the pull request after doing some testing on our computers. Computer 1 ("win8 normal laptop"): Computer 2 ("win8 laptop with touchscreen"): Since the document.ontouchstart seems to be a bit unreliable it might be best to listen to both. At least it works good for us. |
@FinalAngel could you review this quickly? |
@digi604 we can implement this, it is yet not a proper solution for the entire mobile editing topic |
Bugfix for document.ontouchstart detection
No description provided.