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

CommentFileSizeAuthor
#5 todolist.png34.65 KBstborchert
#2 todolist_css_amadou.png44.69 KBChristefano-oldaccount
todolist_css.patch2.58 KBstborchert

Comments

ms2011’s picture

ms2011 commented
Status: Needs review » Fixed

Excellent observation, Stefan! Patch applied to HEAD.

Thanks for your contribution. :)

Christefano-oldaccount’s picture

Christefano-oldaccount commented
Status: Fixed » Active
StatusFileSize
new todolist_css_amadou.png44.69 KB

Todolist looks great and works well in Garland, but its CSS completely destroys some third-party themes like amadou.

stborchert’s picture

stborchert
he/him
Primary language German
commented

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).

Christefano-oldaccount’s picture

Christefano-oldaccount commented

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.

stborchert’s picture

stborchert
he/him
Primary language German
commented
StatusFileSize
new todolist.png34.65 KB

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).

ms2011’s picture

ms2011 commented

@christefano: can you confirm this is still an issue?

marvil07’s picture

marvil07 commented
Status: Active » Fixed

I think this was solved a long time ago, but reopen it if it's needed

Status: Fixed » Closed (fixed)

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