IE 7 Quotes Fix

  • Resolved SFMichael

    (@sfmichael)


    11 years, 11 months ago

    Hi,

    I’m not sure if you are aware, but the before: content doesn’t work in ie 7.

    Luckily because you added a span with a class for the quotes we can add this to fix it.

    .testimonialswidget_open_quote {
    *zoom: expression( this.runtimeStyle[‘zoom’] = ‘1’, this.innerHTML = ‘“’);
    }

    .testimonialswidget_close_quote {
    *zoom: expression( this.runtimeStyle[‘zoom’] = ‘1’, this.innerHTML = ‘”’);
    }

    Can’t add the hyphen the same way though as no span.

    Just thought some one might find this interesting.

    https://www.ads-software.com/extend/plugins/testimonials-widget/

Viewing 6 replies - 1 through 6 (of 6 total)
  • Plugin Contributor Michael Cannon

    (@comprock)

    Thank you for this.

    Can you please let me know if https://gist.github.com/michael-cannon/5255903 works for IE7 regarding quotes. If so, I’ll look into the other content entries for alternatives.

    Thread Starter SFMichael

    (@sfmichael)

    It seems to work just fine. As i mentioned the hypen is also missing in ie 7, this can be fixed by a similar method of using a span for the hypen instead.

    Thanks for adding the code, Excellent plugin.

    Thread Starter SFMichael

    (@sfmichael)

    Hello again,

    I never noticed the .testimonialswidget_join didn’t work either so here’s the fix.

    .testimonialswidget_join {
    *zoom: expression( this.runtimeStyle[‘zoom’] = ‘1’, this.innerHTML = ‘,&nbsp’);
    }

    Plugin Contributor Michael Cannon

    (@comprock)

    Thank you for the compliment and CSS update.

    I’ve looked into the IE7 fixes for the cite::before and I’m wondering about the expression to prepend a span to the cite tag. Any thoughts to doing that?

    Could you please try out https://gist.github.com/michael-cannon/5255903 again regarding join CSS?

    Thread Starter SFMichael

    (@sfmichael)

    I’ll have a think about prepending a span tag to the cite tag.

    As for your latest update you need to remove the “&amp” code only needs to be

    *zoom: expression( this.runtimeStyle[‘zoom’] = ‘1’, this.innerHTML = ‘,?&nbsp’ );

    dont forget to add /remove semi colon following &amp and &nbsp

    Thread Starter SFMichael

    (@sfmichael)

    As for prepending, I havn’t tested this but the jQuery method would be:

    jQuery(‘#YOURID’).prepend(‘<span class=”CLASSNAME”></span>’);

    Also note I’m using jQuery and not $ as i’m assume it would be in no conflict mode.

    Based on a very quick look at the php could you not add this then add the correct css.

    if ( ! empty( $cite ) )
    $cite = ‘<cite><span class=”CLASSNAME”></span>’ . $cite . ‘</cite>’;

    Line 775 / 776

Viewing 6 replies - 1 through 6 (of 6 total)
  • The topic ‘IE 7 Quotes Fix’ is closed to new replies.