Hi there,

This looks like a pretty slick module, unfortunately for me after install it didn't do anything. Specifically, I installed the module, activated it, and gave myself permissions to use it.

I edited my my-theme.info file to include the code snippet you mention on your page. I think expected the conditional statement to appear. However it did not appear. I de-activated the theme, switched to another theme, switched back and reactivated. Still no code.

I am running Drupal 6.10

I uninstalled your module and inserted

<!--[if lt IE 7]>
  <link type="text/css" rel="stylesheet" media="all" href="ie6-and-below.css" />
<![endif]-->
<!--[if IE 7]>
  <link type="text/css" rel="stylesheet" media="all" href="ie7.css" />
<![endif]-->

Into my page.php.tpl file instead, and that worked just fine.

CommentFileSizeAuthor
#3 Conditional_Styles-389988-3.patch333 bytesBWPanda
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

BWPanda’s picture

Did you remember to clear the theme cache after you edited your .info file?

Stoob’s picture

That's an interesting question. Clearing the theme cache is not in the module documentation, so no I did not clear the theme cache. If it is a requirement, the module developers should consider putting that requirement in the description of the module and/or adding documentation for the module. I am not using the module anymore anyway, so I cannot test if clearing theme cache would have had any effect.

BWPanda’s picture

Title: Installed but doesn't work » Add 'clear cache' reminder to documentation
Version: 6.x-1.0 » 6.x-1.x-dev
Component: Code » Documentation
Category: bug » task
Status: Active » Needs review
FileSize
333 bytes

In that case, this should probably be changed to a documentation task.

I checked the readme.txt file but couldn't find any references to clearing the cache, so I created a very simple patch to add this reminder.

JohnAlbin’s picture

Title: Add 'clear cache' reminder to documentation » Add "clear .info cache" reminder to documentation
Status: Needs review » Fixed

The following important note has been added to the README.txt:

Drupal 6 also stores a cache of the data in .info files. If you modify any lines
in your theme's .info file, you MUST refresh Drupal 6's cache by simply visiting
the admin/build/themes page.

Thanks for the suggestion, Peter!

BWPanda’s picture

Glad to help!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.