[wp-trac] [WordPress Trac] #16351: Default Text Widget Creates Invalid HTML

[WordPress Trac]

#16351: Default Text Widget Creates Invalid HTML
--------------------------+-----------------------------
 Reporter:  clwill        |      Owner:
     Type:  defect (bug)  |     Status:  new
 Priority:  normal        |  Milestone:  Awaiting Review
Component:  General       |    Version:  3.0.4
 Severity:  normal        |   Keywords:  Text Widget
--------------------------+-----------------------------
 The default text widget in default-widgets.php creates HTML that is
 invalid.  Older browsers (specifically IE6) stop rendering when they hit
 the construct, disabling the entire rest of the sidebar.

 The issue is that the widget includes a <div> inside a <li>, which renders
 fine (is ignored) in later browsers, but a) doesn't validate, and b)
 causes IE6 to puke, as noted above.

 The specific error is line 382:

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

 Two suggested possibilities:  1) Remove the div entirely or 2) if the
 class is deemed necessary, replace it with a <span
 class="textwidget"></span>

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/16351>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software