Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.As we fixed the fieldset issues for seven we can move on with the next one: Garland in IE8.
This issue is currently fixed for IE6/IE7 using hacks, but we don't have such a (valid) selector "hack" for IE8. Maybe we should instead those hacks set the background on the fieldset-wrapper div (introduced since #676800: Fieldsets break design badly)
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | garland.patch | 4.49 KB | szantog |
| ie8-garland.png | 16.53 KB | casey |
Comments
Comment #1
Jeff Burnz commentedsub
Comment #2
szantog commentedThe problem is the next: There are the solution in ie-fix.css. But this is rendered only in ie6.
In template.php there are:
drupal_add_css(path_to_theme() . '/fix-ie.css', array('weight' => CSS_THEME, 'browsers' => array('IE' => 'lt IE 7', '!IE' => FALSE)));Imho it needs two separate ie files (I'm sure, we will get other error). First, I used the names fix-ie and fix-ie6, but i saw the other core themes, there are ie.css and ie6.css. I think, we need to apply this names consequently, therefore in this patch i renamed it.
Comment #3
thedavidmeister commentedAre ugly fieldsets still an issue in ie11?
I know we all like to bash IE, but it's come a long way in the last 4 years. In light of that, I feel we should target the ie versions a bit more directly, like an ie8.css rather than ie.css.