Previously, local tasks (Tabs) passed through theme('menu_item_link'). Currently they go directly through l(). It is necessary for themes to be able to theme the link in order to do pretty tabs, as it often requires putting markup inside the <a> tag in order to achieve the proper look. This worked in Drupal 5.

This patch restores this functionality.

CommentFileSizeAuthor
local-task-item-theme.patch2.22 KBmerlinofchaos
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented
Status: Needs work » Needs review
chx’s picture

chx CreditAttribution: chx commented
Status: Needs review » Reviewed & tested by the community

It's okay.

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented
Status: Reviewed & tested by the community » Fixed

Thanks, committed.

Anonymous’s picture

(not verified) CreditAttribution: commented
Status: Fixed » Closed (fixed)
adpo’s picture

adpo CreditAttribution: adpo commented
Version: 6.x-dev » 6.5

It is not working for me.
I received a message:

patching file includes/menu.inc
Hunk #1 FAILED at 908
Hunk #2 FAILED at 1121
Hunk #3 FAILED ar 1153