Posted by leafish_paul on November 28, 2005 at 5:39pm
| Project: | Dashboard |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | merlinofchaos |
| Status: | closed (fixed) |
Issue Summary
As the dashboard.css file is imported in hook_view, it can be imported more than once if a site is displaying more than one dashboard on a single page.
Attached patch fixes this by placing the drupal_set_head() call in hook_menu() instead, which I is the correct place. I've also altered the call to get the path to the module/CSS file properly using drupal_get_path() instead of $GLOBALS[base_url].
Note this issue applies to 4.6 version, too: patch on its way.
| Attachment | Size |
|---|---|
| dashboard.module_insert_css_correctly.patch | 1 KB |
Comments
#1
The rest of that post:
Attached patch fixes this by placing the drupal_set_head() call in hook_menu() instead, which I think is the correct place. I've also altered the call to get the path to the module/CSS file properly using drupal_get_path() instead of $GLOBALS[base_url].
#2
Patch applied and committed.
Thank you!
(This points out that no matter how much I think I've figured out about The Right Way to do things in Drupal, there's always something else I didn't notice)
#3
Fastest. Contrib. Commit. Ever.
Nice one!
#4