I replaced every line of hardcoded html with theme functions.

There is no change in functionality or design.

it would be nice if someone could test...

CommentFileSizeAuthor
#2 everything_themeable.aggregatormodule_1.patch6.15 KBnickl
everything_themeable.aggregatormodule.patch6.04 KBTobias Maier

Comments

Tobias Maier’s picture

Tobias Maier commented
Title: replace hardcoded html with theme-functions » aggregator: replace hardcoded html with theme-functions
nickl’s picture

nickl commented
StatusFileSize
new everything_themeable.aggregatormodule_1.patch6.15 KB

Rerolled to fresh head.

Changed one oversite of $output .= to $output = after removal of previous $output binding.

Kjartan’s picture

Kjartan commented
Status: Needs review » Needs work

Sorry the patch doesn't commit nicely anymore. Mind updating it for latest HEAD?

ricabrantes’s picture

ricabrantes commented
Version: x.y.z » 7.x-dev

any news about this?

mustafau’s picture

mustafau commented
Status: Needs work » Fixed

I believe this was fixed long ago.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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