Comments

Status:Active» Needs review
StatusFileSize
new1.29 KB

It has been converted.

With preprocess, I'm thinking something more like this. However, it seems I couldn't find a 'local task' this affects - the primary nav and the tabs for a node, for example, don't seem to be affected by this. Can anyone confirm this patch works for Stark?

Assigned:likin» killerpoke

StatusFileSize
new143 bytes
new2.28 KB

Tested with Stark, works for me except one notice:

Notice: Indirect modification of overloaded element of Drupal\Core\Template\Attribute has no effect in template_preprocess_menu_local_task() (line 1682 of core/includes/menu.inc).

I'v fixed this according to the annotations in Drupal\Core\Template\Attribute.

Assigned:killerpoke» Unassigned

Issue tags:+drupal-at-twig

Add tag

Status:Needs review» Reviewed & tested by the community

Tested killerpoke's patch. PHP notice is gone now and rest looks good.
One small notice: inactive menu tasks have an empty class attribute now, but that's actually a different topic.

Title:Coverting theme_menu_local_taskConvert theme('menu_local_task') to Twig
Assigned:Unassigned» steveoliver
Status:Reviewed & tested by the community» Needs review
Issue tags:-drupal-at-twig+Twig

Updating title, tags; assigning myself.

Status:Needs review» Needs work

Updating status.

Status:Needs work» Fixed

Closing this (instead of moving it to core queue) in favor of aggregated 'menu.inc' core issue #1898478: menu.inc - Convert theme_ functions to Twig. Git attributions to all participants will happen there.

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