Error in use of pngfix.js

sjz - April 4, 2008 - 03:48
Project:Pluralism
Version:6.x-1.0
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

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]-->

#1

bboyjay - April 4, 2008 - 04:16
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.

#2

bboyjay - April 4, 2008 - 04:38
Assigned to:bboyjay» Anonymous
Status:active» closed

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.

 
 

Drupal is a registered trademark of Dries Buytaert.