Implement a generic clearfix

[helen]

We have a clearfix​ defined in core, but it's applied to very specific items. It should be a generic class. We also have an old-school br.clear and a .clear class - should look for those and replace them in core wherever it makes sense.

[afercia]

+1 for this.
A more modern method to contain floated elements would be very useful especially now that core is using a focus style with box-shadows. In many places they're "broken" cause overflow: hidden used to contain floats. See for example: #29897 and #26646. Would suggest:

.contain-floats:after {
    content: '';
    display: table;
    clear: both;
}

Naming convention to discuss of course.

[joemcgill]

For reference: Chris Coyier has kept a pretty good running diary of the various CSS patterns for this here: ​https://css-tricks.com/snippets/css/clear-fix/

[afercia]

thanks @joemcgill. Had a second thought about naming convention, I was wrong. With overflow: hidden still in my mind I was thinking to "contain floats" because that's what overflow: hidden actually does, but with the pseudo-element it's really a "clearfix" :)

[wonderboymusic]

I would love for someone to pick this up - there are divs floating in the breeze all over the place that have no layout, yet contain untold numbers of floated elements

[afercia]

Related #28864 among other things tries to introduce a generic rule for this in common.css:

.contain-floats:after {
	content: '';
	display: table;
	clear: both;
}

[afercia]

The original clearfix mentioned in the ticket description is now in nav-menus.css. On the other hand, [35673] introduced a new rule in common.css, currently used only for the "navigation tabs" container:

/* contain floats */
.nav-tab-wrapper:after {
	content: "";
	display: table;
	clear: both;
}

I'd suggest to just add a generic class name, maybe "wp-clearfix"?

/* modern clearfix */
.wp-clearfix:after,
.nav-tab-wrapper:after {
	content: "";
	display: table;
	clear: both;
}

cc @michaelarestad cause I know he's passionate about CSS naming conventions :)

[michaelarestad]

@afercia You know me so well. I agree that .wp-clearfix is an ideal class name. Once in place, we could remove the need for .nav-tab-wrapper and any other classes using the rules for a clearfix.

[afercia]

Replying to michaelarestad:

we could remove the need for .nav-tab-wrapper and any other classes using the rules for a clearfix.

Thinking probably it's not so easy. For example:

• plugins already use the "Tabs" so removing the clearfix from .nav-tab-wrapper will break their layout since it's unlikely plugins authors are going to add .wp-clearfix
• stuff that can be enqueued in the frontend (e.g. Editor, Media Modals) would need a clearfix rule in a file that's available in the frontend

I'd say that for now the most important thing is introducing the new clearfix and use it for new developments. See proposed patch. Any thoughts more than welcome.

[michaelarestad]

I'd say that for now the most important thing is introducing the new clearfix and use it for new developments. See proposed patch. Any thoughts more than welcome.

Ah. I agree with that. The patch looks good, though makes me a bit sad that we need to keep the extra class in there.

[afercia]

Replying to michaelarestad:

though makes me a bit sad that we need to keep the extra class in there.

Yep :( maybe one more reason to seriously consider @helen 's idea to introduce a deprecated.css

[afercia]

The refreshed patch moves the new clearfix rule to the "utility classes" section.

[afercia]

Thinking it would be better to start using the new CSS class in the cleanest possible way so what about something like 26396.3.patch​ where:

• Tabs output by core will use wp-clearfix
• Tabs output by plugins will fallback to a slightly less-ideal clearfix method overflow: hidden

The CSS would be cleaner and the back-compatibility rules ready to be moved in a deprecated file. Any thoughts more than welcome.

[michaelarestad]

Replying to afercia:

Thinking it would be better to start using the new CSS class in the cleanest possible way so what about something like 26396.3.patch​ where:

• Tabs output by core will use wp-clearfix
• Tabs output by plugins will fallback to a slightly less-ideal clearfix method overflow: hidden

I generally stay away from overflow: hidden because it could have unexpected results. I'm not sure we get a clear benefit from doing this other than completely separating the clearfix. My preference would be to either duplicate the clearfix to keep the clearfix class separate from other classes or just go with something like 26396.2.patch​.

I lean toward the duplication of rules. I'm also wondering if we should start separating out utility classes like wp-clearfix from common.css into their own file.

[afercia]

Replying to michaelarestad:

I lean toward the duplication of rules.

Thanks Michael. OK, wiill refresh the patch.

I'm also wondering if we should start separating out utility classes like wp-clearfix from common.css into their own file.

Good point :)

[afercia]

OK so in order to keep the clearfix class separate from other classes and at the same time be as clean as possible and preserve back-compat I'd propose:

• for modern browsers, use .nav-tab-wrapper:not(.wp-clearfix):after
• for IE 8, no need to be 100% clean so I'd propose to always establish a new block formatting context

See refreshed patch.

Open to suggestions and improvements. Can also be refined later. For now, I'd propose to commit what we have and move on since this is blocking other tickets (see for example #28864).

Will keep this ticket open for further patches to replace the old clearfix and all the br.clear and div.clear. Patches welcome :)

[afercia]

In 36171:

Introduce a new generic CSS clearfix utility class.

.wp-clearfix is now the recommended way to clear and contain floated elements.
Adds back compatibility for the .nav-tab-wrapper navigation tabs.

See #26396.

[afercia]

Patch 26396.5.remove-old-clearfix.patch​ is a first attempt to remove the old clearfix applied to specific items across the admin. Some testing would be nice :)

[paulwilde]

<div id="widgets-right wp-clearfix"> should be <div id="widgets-right" class="wp-clearfix"> in src/wp-admin/customize.php.

[afercia]

@paulwilde thanks! Patch refreshed.

[afercia]

Refreshed patch after recent changes to the markup. Patches like this one tend to stale quickly so I'd propose to commit now and if something breaks I think there's enough time to fix it at this point of the release cycle. By the way, looks good to me.

[afercia]

In 36422:

After [36171] remove all the occurrences of the old CSS clearfix.

The old clearfix was applied to very specific items and defined multiple times
across CSS files. Uses the new generic .wp-clearfix utility class instead.

See #26396.

[lkraav]

What's the ticket for dropping the legacy box model flow breaking floating strategy and moving to modern grid approach like flexbox instead?

[afercia]

Replying to lkraav:

What's the ticket for dropping the legacy box model flow breaking floating strategy and moving to modern grid approach like flexbox instead?

@lkraav there isn't one, as far as I know. Please do feel free to open a new ticket if you think starting a discussion about flexbox is worthy. I'm afraid there's no consensus about the adoption of the flexbox model for core, yet. Also, core still supports IE 8 and 9.

[jorbin]

Calling this complete for 4.5. If there are other changes needed in 4.5, we can reopen it as a task.