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

Mobile Touch Support #164

Closed
davidi1 opened this issue Feb 26, 2018 · 4 comments
Closed

Mobile Touch Support #164

davidi1 opened this issue Feb 26, 2018 · 4 comments

Comments

@davidi1
Copy link

davidi1 commented Feb 26, 2018

Hello, I am having an issue with iOS not handling touch support correctly. Whenever I try to open a study from the main page, it just zooms in on the patient’s name. If I try to long press on the patient’s name, it just sits there and does nothing.

I have tried using Safari and Google Chrome on an iPad and iPhone. Both devices are on iOS 11.

While searching to see if others have had this issue, I came across an older issue (#20) that says it has been fixed. But, for the life of the me, I cannot get it to work.

Am I missing something while compiling? Is it a feature I need to add separately? Or is this a bug? Any help would be much appreciated! If you need anything from me, let me know I will be happy to provide any help I can on my end.

Side note, I did try this on an Android tablet that is running Android 7.1.1 and while touch did bring up a study, it would crash while loading images. I will open a separate issue for this. So, as of right now, it appears to be just an iOS issue.

@swederik
Copy link
Member

swederik commented Mar 1, 2018

We don't really have mobile support as a priority on our roadmap at the moment, so I am sure there are tons of bugs. I'll keep this open until we get to it.

@davidi1
Copy link
Author

davidi1 commented Mar 1, 2018

Sounds good, thank you for letting me know.

@swederik
Copy link
Member

Hi David,

We've fixed the double-tap to zoom issue. It looks like the Hammer.js events were not being caught by handlers defined in the Template.*.events object. I don't know why that stopped working.

@davidi1
Copy link
Author

davidi1 commented Jul 10, 2018

Hello, this is working great now. Thank you!

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

No branches or pull requests

2 participants