Hi. I note that on Nancy's site, the word "Drupal" brings up a hovertip that has a header in it. I'm assuming this is a glossary item, and I've been trying to get my glossary items to have headers too, but it doesn't seem to work; when I put HTML in the glossary item, it just disappears. I'd like some information on this, preferably in the documentation.

Comments

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented
Title: Documentation on Glossary item headers » HTML in term descriptions
Assigned: Unassigned » nancydru
Priority: Normal » Minor

I'd have to do some checking, but I can pretty much guarantee that the version on my site is "older" than the current version. I'm so busy with module maintenance that I have trouble keeping up with all my sites (I even have a couple that are still on 5.1).

Originally, Glossary did a check_plain which disallowed HTML in the term description. At some point, I removed that, but I think it has crept back in for some reason that escapes me at the moment.

wayland76’s picture

wayland76 commented

Can we have it configurable?

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented

Personally I don't want check_plain at all. I looked at it today and you should be able to do what I do.

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented
Status: Active » Fixed

HTML in a term's description works. You do, however, need to make sure that any tags used in the description are allowed in the default input format (usually Filtered HTML).

Docs updated: http://drupal.org/node/201763#html

wayland76’s picture

wayland76 commented
Status: Fixed » Postponed

Not on my site it doesn't. I have a glossary item which displays italics on the glossary page, but the hovertip has no italics. The default filter is one called "MediaWiki", which is based on Flexifilter, but since that's used on the glossary page too, you would think that it would work just fine. However, I'm happy if we postpone this until the hovertips are 100% ported to 6.x.

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented
Status: Postponed » Fixed

Oops, I wish you had said that to start with. I had this fixed once and it got messed up again.

On line 411, you can change "$term_title" to "$term->description" and it will work. I am committing a fix that will roll up at Noon GMT.

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented

Oh, what the heck, I just made a new release with this in it.

wayland76’s picture

wayland76 commented

Agreed, fixed. Thanks a lot!

Anonymous’s picture

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

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