I found two small problems using your module. Other than that it's great! 8-)

The header has no easy to reference class. So I added link-to-us-header. That way I could change it with a red background and a black edge.

Since I'm using a WYSIWYG editor (FCKeditor) the header is being transformed to a rich text area. This means you get tags in there such as <p>, <em>, etc. To support those, all you have to do is use check_markup() instead of check_plain() for the header.

I'm offering a patch for that purpose.

Thank you.
Alexis Wilke

CommentFileSizeAuthor
link_to_us-6.x-1.x.patch512 bytesAlexisWilke

Comments

AlexisWilke’s picture

Assigned: Unassigned » AlexisWilke
Status: Active » Needs review

Missed marking that I was offering a patch here...

Gman’s picture

Patch committed. Thank you.

AlexisWilke’s picture

Status: Needs review » Fixed

Cool, thank you. I'm marking this fixed then. 8-)

Status: Fixed » Closed (fixed)

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