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

use ng-click instead of ng-mousedown for touch devices #112

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

GraemeRFalkner
Copy link

ngQuickDate does not work on mobile devices with the current version of Angular 1.x.
The reason is that Angular does not emit an ng-mousedown when the user taps on a calendar entry.
Two new options have been added, both default to false to prevent breaking changes on legacy installations.
isTouchDevice: Allows ngQuickDate to be manually configured to use ng-click instead of ng-mousedown
detectTouchDevices: Detects if touch is available and uses ng-click or ng-mousedown as appropriate.
Updated version number to 1.3.4

Graeme Falkner and others added 3 commits October 3, 2015 20:15
Add two new options, both of which default to False to prevent any change of behaviour for legacy users.
isTouchDevice: set to true to manually specify touch events
detectTouchDevices: Detect if code is running on a touch device, and use touch events if applicable.
@adamalbrecht
Copy link
Owner

@GraemeRFalkner How do I reproduce this not working? It seems to work on my iPhone without using either of the new options.

@GraemeRFalkner
Copy link
Author

What version of Angular are you using? We found that ngQuickDate worked fine with Angular 1.0 but not with Angular 1.3.4

@adamalbrecht
Copy link
Owner

I still can't reproduce it (on the demo page) with either Angular 1.3.4 or 1.4.7 (the latest) on my iPhone 6 with iOS 9.

It's very possible that it's breaking on other mobile platforms (Android or older versions of iOS) or by using a different set of circumstances from the demo page.

@GraemeRFalkner
Copy link
Author

Ok Thanks, I will investigate further to try to figure out why it doesn't work for us.

@novarac23
Copy link
Collaborator

@GraemeRFalkner Any updates on this ? Free to help out if its needed!

@novarac23
Copy link
Collaborator

@GraemeRFalkner Hey mate! Need any help with that investigation ? I am willing to help out if needed :)

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.

3 participants