-
Notifications
You must be signed in to change notification settings - Fork 892
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
Integration of detection logic Mobile-Detect@2.8.13 ? #28
Comments
The patterns were not up-to-date. I've updated them. Difference: the new patterns recognize 'windows phone' as a mobile device. |
Thanks for updating the "Detect Mobile Browsers" patterns. Is there any reason why you did not add "|android|ipad|playbook|silk" to the first regex? Under http://detectmobilebrowsers.com/about the author states: Tablets |
Yet another valuable input, thanks! a) add this pattern to the first regex, or What do you think is better? |
phuuu, good question. I use version a) for nearly two years now and it works good and it's proven. But maybe version b) might be the better way for upcoming devices in the future !? - At least it's up to you, sorry. |
Thank you for updating the integration of detection logic to Mobile-Detect@2.8.13
I just have a short question. Do you use the current Version "Detect Mobile Browsers" (http://detectmobilebrowsers.com/) for integration which was released at 1 August 2014?
The text was updated successfully, but these errors were encountered: