From 21844843742fed7b08e19511445708e82bcbb112 Mon Sep 17 00:00:00 2001
From: Dirk Schmedding <dirk.schmedding@appandweb.de>
Date: Tue, 7 Feb 2017 10:11:16 +0100
Subject: [PATCH] Added missing isActive if child is active

To have sub menu open, it is required to set the parent element isActive.
---
 Resources/views/layout/macros.html.twig | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/Resources/views/layout/macros.html.twig b/Resources/views/layout/macros.html.twig
index 06d90fab..69dbad11 100644
--- a/Resources/views/layout/macros.html.twig
+++ b/Resources/views/layout/macros.html.twig
@@ -37,7 +37,13 @@
 
 {% macro menu_item(item) %}
     {% if item.route or item.hasChildren %}
-        <li id="{{ item.identifier }}" class=" {{ item.isActive ? 'active' : '' }} {{ item.hasChildren? 'treeview' : '' }}">
+        {% set childIsActive = false %}
+        {% for child in item.children %}
+            {% if child.isActive %}
+                {% set childIsActive = true %}
+            {% endif %}
+        {% endfor %}
+        <li id="{{ item.identifier }}" class=" {{ (item.isActive or childIsActive) ? 'active' : '' }} {{ item.hasChildren? 'treeview' : '' }}">
             <a href="{{ item.hasChildren ? '#': '/' in item.route ? item.route : path(item.route, item.routeArgs) }}">
                 {% if item.icon %} <i class="{{ item.icon }}"></i> {% endif %}
                 <span>{{ item.label }}</span>