[theme-reviewers] Clearing Floats

[Philip M. Hofer (Frumph)]

Coincidently you can do <br class="clear-margins" />

.clear-margins {

clear: both;

height: 0;

line-height: 0;

font-size: 0;

}

and look, it's no longer an empty div.



  ----- Original Message ----- 
  From: Chip Bennett 
  To: theme-reviewers at lists.wordpress.org 
  Sent: Wednesday, September 01, 2010 6:06 AM
  Subject: [theme-reviewers] Clearing Floats


  I've noticed on several reviews, the recommendation to add a "clearfix" <div> beneath the_content()/the_excerpt().


  I'm wondering: should we *really* be making this recommendation? I've always considered an empty div whose sole purpose for existence is to ensure that floats are cleared to be a method of last resort.


  I'm just wondering, since we don't make many other markup-practice recommendations with respect to conformance to the Guidelines. Might it be better simply to state "ensure that floats introduced by the_content()/the_excerpt() are cleared"?


  Chip


------------------------------------------------------------------------------


  _______________________________________________
  theme-reviewers mailing list
  theme-reviewers at lists.wordpress.org
  http://lists.wordpress.org/mailman/listinfo/theme-reviewers
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20100901/fc81b294/attachment.htm>