Posted by stBorchert on June 23, 2007 at 6:20pm
| Project: | Todolist |
| Version: | 5.x-1.x-dev |
| Component: | User interface |
| Category: | bug report |
| Priority: | critical |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
I've tested the module and none of your css-rules is working (and the images aren't shown either).
Why do you build the css file into a php file and do a BOOTSTRAP_FULL? A simple css is working fine (see attached patch). No need to write the complete path to the images :-)
hth,
Stefan
PS: I've replaced todolist.css.php with todolist.css
| Attachment | Size |
|---|---|
| todolist_css.patch | 2.58 KB |
Comments
#1
Excellent observation, Stefan! Patch applied to HEAD.
Thanks for your contribution. :)
#2
Todolist looks great and works well in Garland, but its CSS completely destroys some third-party themes like amadou.
#3
Are you sure its because of todolist.css?
The stylesheet is only loaded and used if you visit a "todolist" node (look at the source of admin/build/themes: it's not there).
I've tested amadou and it looks fine. But todolist isn't displayed correct (remove, edit, move are not shown).
#4
Yes, the CSS gets seriously borked when Todolist is enabled (I had CSS aggregation turned off). Disabling Todolist restored the navigation block, etc. in amadou. I was wondering if I was using a modified version of amadou, so I downloaded a fresh copy and it was still malfunctioning when Todolist is enabled.
#5
Hm, that's really weird.
Could you please check the html source of a page that is broken. "todolist.css" shouldn't be included (if it is, there's an error in the module I didn't see, yet).
#6
@christefano: can you confirm this is still an issue?
#7
I think this was solved a long time ago, but reopen it if it's needed
#8
Automatically closed -- issue fixed for 2 weeks with no activity.