Error in use of pngfix.js

Posted by sjz on April 4, 2008 at 3:48am

Issue Summary

Browser: IE7 does not need pngfix.js loaded and it is imported using the .info file.

Solution: remove the script[] = pngfix.js line from the .info file and, in file page.tpl.php, after the fragment:

  <!--[if gte IE 7]>
  <link type="text/css" rel="stylesheet" media="all" href="<?php print base_path().$directory; ? /ie7fix.css" />
  <![endif]-->

add the following code:

  <!--[if lt IE 7]>
  <script defer type="text/javascript" src="pngfix.js"></script>
  <![endif]-->

Posted by bboyjay on April 4, 2008 at 4:16am

Assigned to:	Anonymous	» bboyjay
Status:	needs review	» active

Actually, the icons I was using that required that script, I no longer use them in this theme (licensing issue).

I really should take that pngfix right out. Thanks.

Posted by bboyjay on April 4, 2008 at 4:38am

Assigned to:	bboyjay	» Anonymous
Status:	active	» closed (fixed)

Done and committed. Since this is not a security bug and really doesn't affect the UI, I won't create a new release yet.