[wp-trac] [WordPress Trac] #21743: Twenty Twelve: add support for Status post format

WordPress Trac wp-trac at lists.automattic.com
Thu Sep 20 19:18:33 UTC 2012


#21743: Twenty Twelve: add support for Status post format
---------------------------+---------------------------
 Reporter:  lancewillett   |       Owner:  lancewillett
     Type:  enhancement    |      Status:  accepted
 Priority:  normal         |   Milestone:  3.5
Component:  Bundled Theme  |     Version:
 Severity:  normal         |  Resolution:
 Keywords:  has-patch      |
---------------------------+---------------------------

Comment (by drewstrojny):

 Replying to [comment:8 lancewillett]:
 > @drewstrojny I think the "clear" class you added is unnecessary. Did you
 see any float clearing problems?

 I added it to make the module itself a bit more bulletproof. If someone
 increases the size of the avatar and adds some extra meta it will still
 clear completely and look good. If the <header> isn't cleared the content
 could start wrapping up against it in that scenario.

 Feel free to move the clear stuff to the stylesheet if you prefer it. I
 know you guys extracted the old clearing method we were using with Sass
 and replaced it with something else. I poked around a bit and found the
 clear class.

 In summary, I think I'd prefer we keep it, but it doesn't have to be in
 the markup.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/21743#comment:10>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list