Salsa icon indicating copy to clipboard operation
Salsa copied to clipboard

IE7 boxsizing.htc being included in IE8 also

Open LeighBicknell opened this issue 12 years ago • 1 comments

Currently in _system.scss (around line 89) the IE7 box-sizing support 'behavior' attribute is also being applied to IE8 and breaking it's layout. Adding a * before 'behavior:' will cause IE8 to ignore this, solving the problem.

I've compiled mine using phpsass and it compiles fine this way, I'm not sure if Compass will compile it properly with the * in there though, so it may be worth checking this first.

LeighBicknell avatar Jan 17 '13 12:01 LeighBicknell

The star hack won't be a problem with compass since compass uses that too. But since IE8 supports box-sizing, the fact that the polyfill breaks the layout sounds to me like a bug in the polyfill. the project page on github clearly says:

I also added feature-detection for box-sizing, to do nothing when detected positive.

It also recommends using the star hack for better performance. and compass does that, so I guess we should too. I just hate shipping IE hacks out-of-the-box. even though they'll be printed only if $ie7-support: true.

tsi avatar Jan 17 '13 12:01 tsi