Closed (fixed)
Project:
Zen
Version:
7.x-5.x-dev
Component:
CSS/SASS/HTML markup
Priority:
Normal
Category:
Bug report
Assigned:
Issue tags:
Reporter:
Created:
25 Apr 2012 at 09:17 UTC
Updated:
4 Jan 2014 at 02:05 UTC
Jump to comment: Most recent
Comments
Comment #1
johnalbinYou mean the shadow is clipped by the parent container?
Comment #2
johnalbinOk. 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.
Comment #3
mustanggb commentedI 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.
Comment #4
johnalbinRight. Good point. And since we can't guarantee the item will have a width, we can't use it. :-\
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. :-\
Comment #5
johnalbinTagging.
Comment #6
johnalbinFixed in the upstream. https://github.com/JohnAlbin/compass-zen-grids/pull/11
Comment #7
johnalbinZen 7.x-5.x now contains Zen Grids 1.1.
Comment #8
mustanggb commentedStirling work good sir
Congrats on a 5.0 stable