WordPress.org

Ready to get started?Download WordPress

Forums

Testimonials by Aihrus
[resolved] IE 7 Quotes Fix (7 posts)

  1. SFMichael
    Member
    Posted 1 year 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.

    http://wordpress.org/extend/plugins/testimonials-widget/

  2. Michael Cannon
    Member
    Plugin Author

    Posted 1 year ago #

    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.

  3. SFMichael
    Member
    Posted 1 year ago #

    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.

  4. SFMichael
    Member
    Posted 1 year ago #

    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');
    }

  5. Michael Cannon
    Member
    Plugin Author

    Posted 1 year ago #

    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?

  6. SFMichael
    Member
    Posted 1 year ago #

    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

  7. SFMichael
    Member
    Posted 1 year ago #

    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

Topic Closed

This topic has been closed to new replies.

About this Plugin

About this Topic

Tags

No tags yet.