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

Not Working in IE 8 at all. #27

Open
cavidano opened this issue Jan 28, 2014 · 4 comments
Open

Not Working in IE 8 at all. #27

cavidano opened this issue Jan 28, 2014 · 4 comments

Comments

@cavidano
Copy link

I love this plugin, but it doesn't work in I.E. 8 at all.

@ghost
Copy link

ghost commented Feb 12, 2014

I had the same problem and figured it out. There's a "console.log" called and that causes an error in IE8. You can easily remove the line in the source code.

@Mithgol
Copy link

Mithgol commented Mar 6, 2014

Should be fixed in bed35fe, please confirm.

@drunkly
Copy link

drunkly commented Nov 5, 2014

Hi, I still had this problem on IE8.

If it help someone I solve checking the elm.css("margin-top") because on IE8 it does parseInt("auto") is NaN instead 0. Then:

top = elm.offset().top - (isNaN(parseInt(elm.css("margin-top"), 10)) ? 0 : parseInt(elm.css("margin-top"), 10)) - offset_top;

Greetings!!.

@Mithgol
Copy link

Mithgol commented Nov 6, 2014

@drunkly Would you make a pull request out of your idea?

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

3 participants