From df710a893bef9b60c600b68925edcc49470635b8 Mon Sep 17 00:00:00 2001
From: webchick <webchick@24967.no-reply.drupal.org>
Date: Wed, 21 May 2014 14:36:52 -0700
Subject: [PATCH] Issue #2271025 by Wim Leers: Don't let
menu_navigation_links() set the active class, the drupal.active-links library
does that for us (plus, it's setting it incorrectly).
---
core/includes/menu.inc | 7 -------
1 file changed, 7 deletions(-)
diff --git a/core/includes/menu.inc b/core/includes/menu.inc
index 0fefe74a004f..27fb20cb15f0 100644
--- a/core/includes/menu.inc
+++ b/core/includes/menu.inc
@@ -464,13 +464,6 @@ function menu_navigation_links($menu_name, $level = 0) {
$class = ' active-trail';
$l['attributes']['class'][] = 'active-trail';
}
- // Normally, l() compares the href of every link with the current path and
- // sets the active class accordingly. But local tasks do not appear in
- // menu trees, so if the current path is a local task, and this link is
- // its tab root, then we have to set the class manually.
- if ($item['link']['href'] != current_path()) {
- $l['attributes']['class'][] = 'active';
- }
// Keyed with the unique mlid to generate classes in links.html.twig.
$links['menu-' . $item['link']['mlid'] . $class] = $l;
}
--
GitLab