From 21dbe4ec47d09c111e81371f2f5234f62626b5ec Mon Sep 17 00:00:00 2001 From: Eric Lippmann Date: Mon, 20 Nov 2017 13:43:54 +0100 Subject: [PATCH] Only render icons of root navigation items The upcoming navigation dashboard creates its links from the menu items in configuration.php. Now, we encourage users to specify icons for second-level menu items but do not want render them in our menu. --- .../Web/Navigation/Renderer/RecursiveNavigationRenderer.php | 3 +++ 1 file changed, 3 insertions(+) diff --git a/library/Icinga/Web/Navigation/Renderer/RecursiveNavigationRenderer.php b/library/Icinga/Web/Navigation/Renderer/RecursiveNavigationRenderer.php index 02653b541..562ed37d6 100644 --- a/library/Icinga/Web/Navigation/Renderer/RecursiveNavigationRenderer.php +++ b/library/Icinga/Web/Navigation/Renderer/RecursiveNavigationRenderer.php @@ -162,6 +162,9 @@ class RecursiveNavigationRenderer extends RecursiveIteratorIterator implements N foreach ($this as $item) { /** @var NavigationItem $item */ if ($item->shouldRender()) { + if ($this->getDepth() > 0) { + $item->setIcon(null); + } if ($this->getUseStandardItemRenderer()) { $renderer = new NavigationItemRenderer(); $content = $renderer->render($item);