Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.IE6.css is not added to the page when I looked at it in IE6 and Firefox
I made a patch based on how Garland adds the IE specific css to the page.
This patch is needed for #569516: D7UX: Make list icons in admin interface clickable
| Comment | File | Size | Author |
|---|---|---|---|
| template.php_.patch | 714 bytes | p.brouwers | |
Comments
Comment #1
rickvug CreditAttribution: rickvug commentedBefore:
<!--[if lt IE 7]><style type="text/css" media="screen">@import 'After:
<!--[if lt IE 7]><link type="text/css" rel="stylesheet" media="all" href="This looks fine for me. Its been a while since debugging IE6, but I do remember it being fussy about how stylesheets are added.
Comment #2
rickvug CreditAttribution: rickvug commentedupdating tags
Comment #3
nkmani CreditAttribution: nkmani commentedApplying the patch, makes following lines now show up in the head section:
Without the patch, these lines are missing (probably because of the wrong key used in seven_process_html() for $vars)
Comment #5
p.brouwers CreditAttribution: p.brouwers commentedpatch failed somehow, resubmitting
Comment #7
roborn CreditAttribution: roborn commented@rickvug yeap, the css works just fine that way.
The problem was the incorrect path to the css, plus there was no html.tpl override to add the $ie_styles var.
This patch fix this issue.
Comment #8
webchickHow I long for the day I can mark this issue "by design"... :\
That code's a bit hard to read with all the various "\n"s in there, but I confirmed that that's how Garland does it as well. The exception was that Garland uses path_to_theme() instead of drupal_get_path(...) so I switched that.
Committed to HEAD. Thanks.