-
Notifications
You must be signed in to change notification settings - Fork 19
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
PolyPage 0.8.3 and IE7 #5
Comments
Eeek! Thanks, I'll take a look into this, this week. |
Any updates on this working with IE7? It does work with IE8 (again, no surprises that IE is wonky), but not IE7. |
this is an issue with triggerEventFromHash in events.polypage.jquery.js replace line 42 with works for me |
sionnnn's change works for me. Thanks sionnnn! |
Just in case anyone else comes across the same issue, I think there should be an underscore after all instances of #pp in sionnnn's code. Github comments seem to strip them out. |
Just downloaded 0.8.3 and tried to run the demo you provided. Anytime I click on one of the triggers at the top, it simply adds "#pp_toggle_admin" to the url. Works in FF3, Opera 9, Chrome, but not IE No surprises there, I guess, but the demo you have posted at http://code.new-bamboo.co.uk/polypage/ does work...
The text was updated successfully, but these errors were encountered: