Hi,
CSS aggregator is removing empty comments like /**/ which apparently is used as an IE6 hack. I have gone through all the bug reports on the CSS aggregator and have not found any mention. Appologies if this is a repetition.

Here is an example CSS block that is being affected:

.foo h2
{
    background-image: url("/sites/all/themes/sample_theme/images/sample_h2_background.png");
    height: 0px !important;
    height /**/:43px;
}

I had to modify the core to get past it :

Index: common.inc
===================================================================
--- common.inc  (revision 78)
+++ common.inc  (working copy)
@@ -2060,6 +2060,11 @@
     $keep_nextone = FALSE;
     return $matches[0];
   }
+
+  if ($matches[0] == '/**/') {
+    return $matches[0];
+  }
+
   switch (strrpos($matches[0], '\\')) {
     case FALSE :
       // No backslash, strip it.

Not sure if this is a good solution or if it has any side-effect.

Thanks.

Comments

Status: Active » Closed (outdated)

Automatically closed because Drupal 6 is no longer supported. If the issue verifiably applies to later versions, please reopen with details and update the version.