WordPress.org

Ready to get started?Download WordPress

Forums

Jetpack by WordPress.com
JavaScript Error in IE7 - Gravatar Hovercards (8 posts)

  1. Curtiss Grymala
    Member
    Posted 1 year ago #

    I am using a self-hosted WordPress site with JetPack enabled. Today, I came across an issue with the Gravatar Hovercards component within JetPack on that site. Within the JavaScript for Gravatar Hovercards (the gprofiles.js?ver=2012Octaaa file, to be specific - it doesn't seem to appear in the version without the ver parameter specified; but it does seem to show up in any copy with "aaa" in its ver parameter), there is an object declaration that ends with a comma. In Internet Explorer 7 and earlier (I've heard reports that this issue is also a problem in IE8, but haven't been able to reproduce that), object declarations that ended with commas threw JavaScript errors, because IE expected one more item to be defined within that object declaration.

    I realize IE7 is old, and warrants very little support, but this type of fix should be fairly simple to correct. If you search gprofiles.js?ver=2012Octaaa, you will find the following statement:

    'padding':'0px','margin':'-2px 0 0 -2px',};if(jQuery.browser.msie&&9>jQuery.browser.version)

    In the CSS declaration found in that snippet, the comma after -2px' is throwing a JavaScript error in IE7. Thanks.

    http://wordpress.org/extend/plugins/jetpack/

  2. Tim Moore
    Moderator
    Plugin Author

    Posted 1 year ago #

    Thanks for letting us know. We'll take a look.

  3. Thomas McMahon
    Member
    Posted 1 year ago #

    Thanks Curtiss for post a fix. Now I can patch my code while I wait for JetPack to update.

  4. tomaszs
    Member
    Posted 1 year ago #

    It would be nice to have this fixed.
    The error also occurs in IE8

  5. Thomas McMahon
    Member
    Posted 1 year ago #

    Can this be patched in the plugin? Or does it have to be fixed on gravatar.com?

  6. Tim Moore
    Moderator
    Plugin Author

    Posted 1 year ago #

    This has been patched on our servers. It may take a few days for the caches to clear out, though.

  7. Thomas McMahon
    Member
    Posted 1 year ago #

    Thank you.

  8. retrocool
    Member
    Posted 1 year ago #

    I still get two script errors on my site when using IE7 and Jetpack and none when I turn Jetpack off.

    Barring IE7 magically becoming a decent browser, I have to turn jetpack off for now.

Topic Closed

This topic has been closed to new replies.

About this Plugin

About this Topic