Default Text Widget HTML Breaks Layout in IE6, IE7 & IE8

  • Since version 3.0.2 there is this nagging bug in default-widgets.php file with the Text widgets class (Line 383) Using the DIV to wrap the LI group makes IE brainfart in the nested “Primary” widget area.

    The issue is that IE doesn’t like the UL > LI > DIV > LI implementation. And it forces IE to read a closing UL prematurely and breaks the HTML.

    Every time I upgrade I have to go in and edit this line to fix the display issue:

    From:
    <div class="textwidget"><?php echo $instance['filter'] ? wpautop($text) : $text; ?></div>

    To:
    <ul class="textwidget"><?php echo $instance['filter'] ? wpautop($text) : $text; ?>

    Can this fix be included in the next patch?

Viewing 7 replies - 1 through 7 (of 7 total)

  • Really, this is broken, can this get fixed, please?

    Where do I find “Text widgets class (Line 383)”?

    Thanks!

    Thread Starter taylorehat

    (@taylorehat)

    WordPress Main Directory > wp-includes > default-widgets.php (Line 383)

    It is part of the Text Widget Class

    Currently it is wrapping the a set of
    <li>‘s with a <div>. IE doesn’t like the order and I am assuming it is picking up the closing </div> and closing the widget prematurely, breaking the layout.

    Good luck!

    Thanks taylorehat, but I am still having the text widget loading below everything on cornellsrestaurant.com after applying the change on IE 6.

    IE 6 should be outlawed at this point!

    Moderator Ipstenu (Mika Epstein)

    (@ipstenu)

    ?????? Advisor and Activist

    taylorehat – Next time, PLEASE wrap your LI and DIVS in code tags ?? You borked the layout of the page ??

    Honestly, I’ve never seen this be a problem before. Report it on trac, but I’m pretty surprised.

    Thread Starter taylorehat

    (@taylorehat)

    up2early: Try wrapping the html content your placing in your text widget with an li tag. It looks like you’ve got this:

    <ul class="textwidget">

... Your table and Facebook widget html you've added in WP Admin.

</ul>

    try this…

    <ul class="textwidget">

<li>
... Wrap Your table and Facebook widget html you've added in WP Admin within an LI.
</li>

</ul>

    Maybe IE6 will like it better?

    lpstenu: Sorry, I totally forgot to wrap my html with code tags. Also, when I posted this here I assumed it was posted to trac. I will have to do that.

    taylorehat:thanks for the suggestion but adding the
    <li> and </li>
    tags in WP admin did not change anything in IE6.

    Thanks for all your help, very much appreciated to this obvious newb.

Viewing 7 replies - 1 through 7 (of 7 total)
  • The topic ‘Default Text Widget HTML Breaks Layout in IE6, IE7 & IE8’ is closed to new replies.

Tags