-
Notifications
You must be signed in to change notification settings - Fork 169
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
IE11 not working #22
Comments
I apologize. I have no access to IE10 or IE11. Do you have the user agent string? I have seen these by doing some searches: IE10: IE11 |
No need to apologize, it's alreay great that you share it 👍 |
Written in PHP, but the regular expression itself should also work in JS. It matches all IE versions, including IE11. |
fix issue Issue rafaelp#22 Signed-off-by:Eric Villard <dev@eviweb.fr>
Hi, if someone have time to review this patch ;) Thanks |
Yes, I will take a look. I myself have no access to IE10 or IE11. But I will take a look at this and integrate when I can. From: eviweb notifications@github.com Hi, |
FWIW I have just tested the above commit on my IE11 and it works for me. |
Where is the updated code for ie 11? |
IE 11 can be tested with: http://stackoverflow.com/questions/21825157/internet-explorer-11-detection |
This should work for detecting IE6-11 and potentially beyond. |
I tested it for ie11 on windows 7.
But I get a red window instead of a yellow window
The text was updated successfully, but these errors were encountered: