Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
Seven theme
Priority:
Normal
Category:
Bug report
Assigned:
Issue tags:
Reporter:
Created:
15 Oct 2009 at 15:10 UTC
Updated:
3 Jan 2014 at 00:29 UTC
Jump to comment: Most recent
Comments
Comment #1
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 commentedupdating tags
Comment #3
nkmaniApplying 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 commentedpatch failed somehow, resubmitting
Comment #7
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.