[beta5] Handle target/title in footer menu links

Hi @mh_nichts

Thanks a lot fo this MR.

I will have a look soon. I guess I will suggest you a solution without preprocess hook because we are targeting a front-friendly/PHP-free codebase.

Indeed, we need a preprocess because UrlHelper is not available in Twig template (and that's a good thing)

However, we can make it simpler:

1. item.target & item.title_attr doesn't exist in Drupal menu structure: https://api.drupal.org/api/drupal/core!modules!system!templates!menu.htm...

Can you stick with item.attributes instead of adding those 2 properties in the preprocess?

So, we can just do that in Twig:

<a{{ item.attributes.addClass("fr-footer__{{ variant }}-link").setAtttribute("href", item.url) }}>

(not 100% sure about my proposal because I don't know if item.attributes is suitable for A element instead of LI element)

2. Once it is done, this can be moved to the Twig template:

if (isset($item['target']) && $item['target'] === '_blank') {
 $item['title_attr'] = t('@title - new window', ['@title' => !empty($item['title_attr']) ? $item['title_attr'] : $item['title']]);
}

Something like that (non tested proposal):

{% set item_attributes = item.attributes %}
{% if item_attributes.hasAttribute('target') and item_attributes.offsetGet('target') == '_blank' %}
  {% set title = item_attributes.hasAttribute('title') ? item_attributes.offsetGet('title') : item.title %}
  {% set item_attributes = item_attributes.setAttribute('title", '@title - new window'||t({@title', title}) %}
{% endif %}

(not 100% sure about my proposal because I don't know if item.attributes is suitable for A element instead of LI element)

See how https://www.drupal.org/project/menu_link_attributes & the new setting type are doing that

Hi Marie,

In the soon-to-be-released "links" (previously "menu" but shared with breadcrumbs and pagers) setting type we have introduced an new property for item, link_attributes:

attributes: HTML attributes for the menu item.
below: The menu item child items.
title: The menu link title.
link_attributes: HTML attributes for the link
url: The menu link URL as string
localized_options: Menu link localized options.
is_expanded: TRUE if the link has visible children within the current menu tree.
is_collapsed: TRUE if the link has children within the current menu tree that are not currently visible.
in_active_trail: TRUE if the link is in the active trail.

#3345071: Add links setting type is not merged yet, so we are open to suggestion if you prefer to manage those attributes differently.

FYI, related issue: #3382940: [beta5] RE: Missing active trail inside side menu pattern (lvl.3)

Thats' great. Merged.