ie conditional styles in .info is awesome, nice work, John!
I use a combo of a -lte IE 7- to catch styles for both, then follow with a -lte IE 6- since most ie hacks are for 6, and then I don't have to use * html.

If in my subtheme .info file, I use any syntax other than [if IE], a duplicate call to ie.css doesn't override the ie.css in the base theme. It will use all 3.

In other words, when I use -
conditional-stylesheets[if lte IE 7][all][] = ie.css
conditional-stylesheets[if lte IE 6][all][] = ie6.css

all 3 are loaded - ie.css from the base theme, plus ie.css and ie6.css from the subtheme.
It's not really a problem, but would rather not load an extra file, and I assume zen will eventually change to not include ie 8.

The only way I could not load the ie.css from the base theme, was to use exactly the syntax [if IE]. I assume this version of -template.conditional-styles.inc- makes this that specific.

I entered this as a bug rather than a feature request, only since you noted "Putting it in the .info file allows sub-themes to replace/remove the styles in the base theme".

Comments

johnalbin’s picture

johnalbin
they/他
Primary language English
Location Taipei, Taiwan
commented
Category: bug » support
Priority: Minor » Normal
Status: Active » Postponed (maintainer needs more info)

Good observation!

Actually, that behavior is by design. Stylesheets will only override a base theme's stylesheet if both the filename and the condition match.

However, try this in the subtheme:

; Prevent the base theme's ie.css from loading. Note: subtheme/ie.css doen't exist.
conditional-stylesheets[if IE][all][] = ie.css
; Load the subtheme's styles
conditional-stylesheets[if lte IE 7][all][] = ie7.css
conditional-stylesheets[if lte IE 6][all][] = ie6.css
echoz’s picture

echoz commented
Status: Postponed (maintainer needs more info) » Closed (fixed)

Of course, that works, closing... thanks John.