[theme-reviewers] Clearing Floats

[Edward Caissie]

As a suggestion ... have a look through Twenty Ten.

The "clear: both;" property is used in several CSS elements with no "empty
div" that I noticed (day job doesn't let me really look for long periods of
time)

On Wed, Sep 1, 2010 at 9:28 AM, Chip Bennett <chip at chipbennett.net> wrote:

> p.s. Phil - regarding specific markup recommendations: this is *exactly*
> the kind of thing that would be great for your Theme Review "Knowledge
> Base".
>
> Chip
>
>
> On Wed, Sep 1, 2010 at 8:27 AM, Chip Bennett <chip at chipbennett.net> wrote:
>
>> Semantically, a parent container around the_content()/the_excerpt() would
>> be more correct. e.g.
>>
>> <div class="post-content">
>> ...the_content()/the_excerpt
>> </div>
>>
>> .post-content {
>>      clear:both;
>> }
>>
>> But again: I don't think we should be making *any* specific markup
>> recommendations regarding how the guideline is achieved. Simply state that
>> the markup must ensure that floats introduced via
>> the_content()/the_excerpt() must be cleared. (IMHO)
>>
>> Chip
>>
>>
>> On Wed, Sep 1, 2010 at 8:24 AM, Philip M. Hofer (Frumph) <
>> philip at frumph.net> wrote:
>>
>>>  Nevertheless it needs to be done after the_content() is displayed, ya
>>>
>>> - Phil
>>>
>>>
>>> ----- Original Message -----
>>> *From:* Edward Caissie <edward.caissie at gmail.com>
>>> *To:* theme-reviewers at lists.wordpress.org
>>> *Sent:* Wednesday, September 01, 2010 6:22 AM
>>> *Subject:* Re: [theme-reviewers] Clearing Floats
>>>
>>> I use the "empty div" method myself ... either with inline style or with
>>> class="clear" or "clear-float".
>>>
>>> My preference would be to use the latter with the more obvious
>>> class="clear-float" element.
>>>
>>>
>>> Cais.
>>>
>>> On Wed, Sep 1, 2010 at 9:15 AM, Philip M. Hofer (Frumph) <
>>> philip at frumph.net> wrote:
>>>
>>>>  The <br method is considered a hack.  while the <div is not.
>>>>
>>>> - Phil
>>>>
>>>>  ----- Original Message -----
>>>> *From:* Chip Bennett <chip at chipbennett.net>
>>>> *To:* theme-reviewers at lists.wordpress.org
>>>>   *Sent:* Wednesday, September 01, 2010 6:13 AM
>>>> *Subject:* Re: [theme-reviewers] Clearing Floats
>>>>
>>>> I like this approach better (if we need to make specific recommendations
>>>> at all.
>>>>
>>>> And, regarding the matter being only "semantics" - well, yeah; that's
>>>> the point. Markup is *supposed* to be semantic.
>>>>
>>>> Chip
>>>>
>>>> On Wed, Sep 1, 2010 at 8:09 AM, Philip M. Hofer (Frumph) <
>>>> philip at frumph.net> wrote:
>>>>
>>>>>  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 <chip at chipbennett.net>
>>>>> *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
>>>>>
>>>>>
>>>>> _______________________________________________
>>>>> theme-reviewers mailing list
>>>>> theme-reviewers at lists.wordpress.org
>>>>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>>>>
>>>>>
>>>>  ------------------------------
>>>>
>>>> _______________________________________________
>>>> theme-reviewers mailing list
>>>> theme-reviewers at lists.wordpress.org
>>>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>>>
>>>>
>>>> _______________________________________________
>>>> theme-reviewers mailing list
>>>> theme-reviewers at lists.wordpress.org
>>>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>>>
>>>>
>>>  ------------------------------
>>>
>>> _______________________________________________
>>> theme-reviewers mailing list
>>> theme-reviewers at lists.wordpress.org
>>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>>
>>>
>>> _______________________________________________
>>> theme-reviewers mailing list
>>> theme-reviewers at lists.wordpress.org
>>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>>
>>>
>>
>
> _______________________________________________
> 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/6bc8aec1/attachment-0001.htm>