While trying to theme my user profile pages with 5 different Content Profile types, I found that the function content_profile_show_profiles on line 479 of the Feb 03 dev release failed to wrap the "Create your links in the prefix and suffix tags (as defined on lines 510 and 511 respectively) if they came after a content type that existed. While trying to figure out why, and make them more CSS addressable, it became easier to just change the way these are handled with div tags. So, to that end, I've attached a set of patches of my changes for your review.

Comments

fago’s picture

Category: feature » task

+1 for better themability, but please also note #421980: remove HTML from .module code.
So shouldn't we try to remove any html out of theme functions?

fred0’s picture

Yes, I would agree with the idea of removing html from the module.