Closed (fixed)
Project:
Typogrify
Version:
5.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Anonymous (not verified)
Created:
17 Jul 2007 at 08:42 UTC
Updated:
25 Mar 2017 at 20:41 UTC
Typogrify uses the wrong Drupal command to add its CSS to Drupal output:
drupal_set_html_head('<style type="text/css" media="all">@import "' . base_path() . drupal_get_path('module', 'typogrify') .'/typogrify.css' . '";</style>');
This interferes with CSS aggregation provided by Drupal 5, and prevents me from overriding Typogrify's default declarations.
Comments
Comment #1
Tim Gambell commentedHi bangpound,
According to http://www.lullabot.com/articles/how_to_properly_add_css_files the way to add css is:
I can't seem to get this to work. If there is another, better way to add CSS, please let me know. In the meantime, I'll root through the docs and project pages and see if I can find some other modules that are trying to do the same thing I am.
Thanks for all your help!
Tim
Comment #2
Tim Gambell commentedComment #3
Tim Gambell commentedLooks like the correct way for a module to add css is to say:
See http://api.drupal.org/api/HEAD/function/drupal_add_css for reference. Will implement asap.
We should allow people to turn this stylesheet off from the admin interface (but it should default to on).
Comment #4
Tim Gambell commentedPatched. Let me know how this works for you. Thanks!
Comment #5
Tim Gambell commentedComment #6
(not verified) commented