When I enable CSS optimization on Drupal 6.16 based site I get "white screen of death" and timeout on drupal_add_css($m.'/css/jquery_ui_dialog-dialog/jquery_ui_dialog.css');

Comments

EugenMayer’s picture

Hi, when you load the jquery_ui_dialog modul or when you try to optimze?

Razunter’s picture

When http://drupal.org/project/wysiwyg_imageupload tries to load it. Only on WYSIWYG-enabled pages and only with CSS compression on.

EugenMayer’s picture

Hmm, no idea. We probably need some help here as iam not very familiar with the css optimizer.

Dirk’s picture

Are you using a Zen based theme? If so, you may have run into the same problem i did. Here is the relevant bug report, there is a work-around in post #7 o that thread.

Razunter’s picture

No, I'm using my own theme, but the problem looks similar
Cleaned CSS file with http://www.codebeautifier.com (standart settings) and now everything works fine

EugenMayer’s picture

@5 Razor which CSS file did you clean? The one from jquery_ui_dialog?

Razunter’s picture

jquery_ui_dialog.css

EugenMayer’s picture

Status: Active » Fixed

Done that also, will check that in. Thank you.

EugenMayer’s picture

Status: Fixed » Closed (fixed)
Razunter’s picture

Version: 6.x-1.0 » 6.x-1.2
Status: Closed (fixed) » Active

v1.2 - same bug

EugenMayer’s picture

Well iam using ther CCS-Optimizer and have no problems, on all installations.

How can you be sure, that this is related to the css of jquery_ui_dialog?

Razunter’s picture

When I comment drupal_add_css("$m/css/jquery_ui_dialog-dialog/jquery_ui_dialog.css"); everything loads fine
Cleaning comments in this CSS-file helps too...
I think it's Drupal core bug, but who knows when it'll be fixed.

Странно это всё :(

EugenMayer’s picture

Ok i will stuff that CSS file through the code-optimizer once again.

Razunter’s picture

No problems with cleaned version: http://dl.dropbox.com/u/103461/jquery_ui_dialog.css

EugenMayer’s picture

Status: Active » Needs review
EugenMayer’s picture

i see, that was cleaning up the jquery ui theme css. Well iam not totaly happy that this is needed, because if the users generates his own theme he will need this cleanfix once again but any, that way it will work out of the box for 95% of all the users.

Thanks for the feedback, will be included in 1.3

EugenMayer’s picture

Status: Needs review » Fixed
EugenMayer’s picture

Status: Fixed » Closed (fixed)