Is there anything in 7.x that would prevent the path to a custom CSS file from working? When I set my path to %t/css/print.css, the styles are not read, but if I overwrite the module's print.css with that file and delete the path from the settings, then things are styled the way I want.

I found an old issue saying you needed to use %b at the beginning of the path, but this issue has clearly been fixed because if you attempt to add that macro you get an error (and also the instructions now tell you the path should be relative to the base.)

Am I crazy? I can't get this to work.

CommentFileSizeAuthor
#5 print-1169148-5.patch496 bytesbradjones1
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

candelas’s picture

the same happens with 6.20
thanks for the module.

jobe12’s picture

The same issue
I used this patch and everything worked http://drupal.org/node/1110046#comment-4341870. It fixed my print.css issue.

candelas’s picture

thanks @jobe12 for the link to the path.
i will try it tomorrow to see if i can fix the problem in my site :)
i am in 6.20.
which version do you use?

jobe12’s picture

i am using 7. i think you should check issues for your drupal version.

bradjones1’s picture

Status: Active » Needs review
FileSize
496 bytes

Also experiencing similar behavior - without a custom css variable set, the module's takes over as expected. However neither a hard-coded path to the theme nor the %t placeholder appears to work.

This is because if the custom CSS is in a theme directory - which most probably are - it was added in and then immediately stripped along with all the other theme stylesheets. I've just added in an AND to the filter to ensure it's not stripping that custom file we just added. Here's a patch against HEAD.

whitefluffy’s picture

Status: Needs review » Reviewed & tested by the community

I had the same problem, and the patch by bradjones1 solved the problem. Works great for me.

Using 7.x-1.x-dev version.

jcnventura’s picture

Status: Reviewed & tested by the community » Fixed

@bradjones1: thanks a lot for the patch.

Unfortunately, the proper solution to this problem is to use the declared CSS group, but your patch reminded me that this quick hack was not the solution (I think I wrote this before CSS groups got introduced in Drupal core)

Anyway, this should now be fixed in dev.

bradjones1’s picture

CSS groups didn't even come to mind, probably because it's so new! Regardless, glad this prompted a fix/re-write nonetheless.

Status: Fixed » Closed (fixed)

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