Grid units should have box-sizing: border-box by default. about pure HOT 5 CLOSED

Since

By default, grid units don't have any margin/padding. If you want to add these, put them in a child container.

there is no need for that

from pure.

+1 for looking into to this. @msweeney it would be great to get your thoughts on this too.

from pure.

I'm assuming the rationale for overriding default box-sizing is so that borders and padding can be applied directly to units, but it would be good to call out the actual issue this resolves.

Pros:

• can add padding + borders to units

Cons:

• lose legacy IE support (box-sizing is IE8+)
• added specificity makes it harder to override/unset box-sizing

I'd prefer we didn't have an opinion either way, and just use the default, assuming users can customize as needed.

from pure.

I'd prefer we didn't have an opinion either way, and just use the default, assuming users can customize as needed.

Yeah, good idea, that's seems like the smart thing to do. @tilomitra we could handle this through documentation, e.g. on the site's grids page we could so an example that uses box-sizing and one which doesn't, describing the pro/cons that @msweeney listed.

from pure.

@ericf @msweeney I'm okay with this. Closing this out and adding it as a documentation issue on pure-site:
https://github.com/yui/pure-site/issues/85

from pure.