Would it be reasonabe to request that the menu_html module be merged into the menu_attributes module?

http://drupal.org/project/menu_html

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

amateescu’s picture

Status: Active » Postponed

Of course it's reasonable but I can't do anything in this regard until someone posts a patch here that I can review.

drupov’s picture

Version: 7.x-1.0-rc2 » 7.x-1.x-dev
Assigned: Unassigned » drupov
Status: Postponed » Needs review
FileSize
3.69 KB

Here's a patch I that makes use of the code in menu_html module and provides a checkbox to enable html in menu link titles.

drupov’s picture

Is there a chance to have that patch commited? It seems to be working pretty stable on my site.

Anonymous’s picture

Issue summary: View changes

Yes I agree very usefull, but I think it would be better to put the extra html in a seperate field. Maybe it breaks the translation for example, but still usefull thanks!

MediaFormat’s picture

Status: Needs review » Needs work

Patch no longer applies

arv.drupal’s picture

Updated patch.

apaderno’s picture

The patch in #6 doesn't seem pertinent to this issue.

apaderno’s picture

Assigned: drupov » Unassigned
apaderno’s picture

Issue tags: +Needs reroll
drupov’s picture

Re-worked the patch from #2, so it applied cleanly against latest version.

Also agree with @apaderno, #6 is not looking like it is related to the issue.

apaderno’s picture

Status: Needs work » Needs review
Issue tags: -Needs reroll