This is due to using the "overflow: hidden;" clearfix method, I have tested jstoller's overflow friendly method and it works perfectly. I would even go so far as to say it should be the method used by default.

Can we get this into Zen Grids 1.0 before release please?
https://github.com/JohnAlbin/compass-zen-grids/pull/11

Comments

johnalbin’s picture

You mean the shadow is clipped by the parent container?

johnalbin’s picture

Ok. This is a deceptively simple change, but it has massive implications.

Changing the clearfix method has subtle, but important changes to Zen Grids' containers. Compass' default clearfix method adds block formatting context to our containers. Changing to a different clearfix method means the container may not have the block formatting context anymore.

http://thinkvitamin.com/design/everything-you-know-about-clearfix-is-wrong/ explains it best.

I wonder if display:inline-block can be used.

mustanggb’s picture

I think "display: inline-block;" is fine if we can guarantee to always specify a width.

Also what is the issue with using the previous method, i.e.

/**
 * Markup free clearing.
 *
 * @see http://perishablepress.com/press/2009/12/06/new-clearfix-hack
 */
.clearfix:after {
  content: ".";
  display: block;
  height: 0;
  clear: both;
  visibility: hidden;
}
/* IE6 */
* html .clearfix {
  height: 1%;
}
/* IE7 */
*:first-child + html .clearfix {
  min-height: 1%;
}
johnalbin’s picture

I think "display: inline-block;" is fine if we can guarantee to always specify a width.

Right. Good point. And since we can't guarantee the item will have a width, we can't use it. :-\

what is the issue with using the previous method

Technically nothing. I was having some odd margin issues with it. But I figured out why that was happening after reading the thinkvitamin article above (see demo #1).

When I converted all the stylesheets to Compass, I just started using its clearfix mixin. It fixed the margin issues I was having. But I didn't notice the problems you mentioned. That is definitely a bug with them.

It looks like we need to use this clearfix technique: http://nicolasgallagher.com/micro-clearfix-hack/

Compass has the pie-clearfix mixin which is "adapted" from the above technique. Unfortunately, they seemed to have adapted away the margin fix I need. :-\

johnalbin’s picture

Issue tags: +7.x-5.0 blocker

Tagging.

johnalbin’s picture

Assigned: Unassigned » johnalbin
johnalbin’s picture

Status: Reviewed & tested by the community » Fixed

Zen 7.x-5.x now contains Zen Grids 1.1.

mustanggb’s picture

Stirling work good sir
Congrats on a 5.0 stable

Automatically closed -- issue fixed for 2 weeks with no activity.