[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