- Issue created by @SKAUGHT
- Merge request !255Issue #3441615. remove legacy template var (and service call) in main hook_theme → (Open) created by SKAUGHT
- Status changed to Needs review
7 months ago 5:30pm 17 April 2024 - 🇪🇸Spain ckrina Barcelona
Moving to Needs work after getting Navigation into core.
- Status changed to Needs work
7 months ago 10:44pm 26 April 2024 - First commit to issue fork.
- Merge request !7785Issue #3441615: navigation_theme() loads 'extension.list.module' for no purpose. → (Open) created by merilainen
- Status changed to Needs review
7 months ago 9:05am 27 April 2024 - Status changed to RTBC
7 months ago 9:19am 27 April 2024 - 🇫🇮Finland kekkis Pirkkala
RTBC +1.
Also checked at least that `path` does not appear anywhere in the templates as a variable. The only occurrence in menu templates of this theme hook refers to the twig function called
path().$ grep -r 'path' --include=menu-region--footer.html.twig ./core/modules/navigation/templates/menu-region--footer.html.twig: attributes: create_attribute({ 'href': path('help.main'), 'data-drupal-tooltip': 'Help'|t, 'data-drupal-tooltip-class': 'admin-toolbar__tooltip' }), - Status changed to Downport
7 months ago 9:41am 27 April 2024 - 🇬🇧United Kingdom catch
Committed/pushed to 11.x, thanks!
Moving to 10.3.x for backport assuming that the main navigation commit gets backported there too.
- Status changed to Fixed
7 months ago 9:29pm 29 April 2024 Automatically closed - issue fixed for 2 weeks with no activity.