Adding a class and WYSIWYG problem
AlexisWilke - January 5, 2009 - 05:07
| Project: | Link To Us |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | AlexisWilke |
| Status: | closed |
Jump to:
Description
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
| Attachment | Size |
|---|---|
| link_to_us-6.x-1.x.patch | 512 bytes |
#1
Missed marking that I was offering a patch here...
#2
Patch committed. Thank you.
#3
Cool, thank you. I'm marking this fixed then. 8-)
#4
Automatically closed -- issue fixed for two weeks with no activity.