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

Fix browser detection to make dropkick work in IE 10 #88

Merged
merged 1 commit into from
Jul 17, 2013

Conversation

steve-v-jedo
Copy link
Contributor

The code as it stands reject IE10 as it thinks that it is a version < 7.

There seems to be an issue with rounded corners not working in IE10, but I couldn't see anywhere where border-radius was being set on a browser by browser basis. Perhaps an IE10 bug?

jonathanselander added a commit to jonathanselander/DropKick that referenced this pull request Dec 16, 2012
Fix browser detection to make dropkick work in IE 10
@nym
Copy link

nym commented Feb 6, 2013

This really needs to get fixed asap.

@babajem
Copy link

babajem commented Mar 17, 2013

I try this and it seems to work:

Modify the lign 16:
if ($.browser.msie &amp;&amp; $.browser.version.substr(0, 1) < 7) {
to;
if ($.browser.msie &amp;&amp; parseInt($.browser.version) < 7) {

@ozinepank
Copy link

Thank @babajem 👍

@jprabhufacebook
Copy link

Thanks much. it is working for IE 10.

@Robdel12 Robdel12 merged commit 16eaa3e into Robdel12:master Jul 17, 2013
@Robdel12
Copy link
Owner

Merged in!

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.

None yet

6 participants