I thought it would be a nice feature to have this filter also modify existing abbr tags that are missing a title attribute.

For example:

<abbr>HTML</abbr>

would be converted to

<abbr title="HyperText Markup Language">HTML</abbr>
CommentFileSizeAuthor
#4 abbrfilter-DRUPAL-6--1_regex_comment.patch1.04 KBdboulet
#1 abbrfilter-DRUPAL-6--1_769428_existing_abbr.patch1.3 KBdboulet

Comments

dboulet’s picture

dboulet commented
Status: Active » Needs review
StatusFileSize
new abbrfilter-DRUPAL-6--1_769428_existing_abbr.patch1.3 KB

This patch adds the feature described above. It also re-formats the pattern to be more readable.

erykmynn’s picture

erykmynn commented
Status: Needs review » Fixed

I incorporated this with the fix highlighted by your bug report.
http://drupal.org/cvs?commit=358192

Should be shortly as a beta release. Apologies for not writing your username in the correct place in the commit. Your patches and feedback were greatly appreciated!

dboulet’s picture

dboulet commented

Great, thanks erykmynn!

dboulet’s picture

dboulet commented
Status: Fixed » Needs review
StatusFileSize
new abbrfilter-DRUPAL-6--1_regex_comment.patch1.04 KB

One little comment nitpick : ).

erykmynn’s picture

erykmynn commented

will add next time around