Skip to content

Commit f885314

Browse files
author
Cari Spruiell
committed
MAGETWO-35081: Backend navigation Menu contains groups with single sections which have titles
- removed level 1 headings when there is only one level 1 group - updated expected results for corresponding test
1 parent 385f475 commit f885314

File tree

2 files changed

+25
-16
lines changed
  • app/code/Magento/Backend/Block
  • dev/tests/integration/testsuite/Magento/Backend/Block/_files/menu

2 files changed

+25
-16
lines changed

app/code/Magento/Backend/Block/Menu.php

Lines changed: 24 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -454,21 +454,30 @@ public function renderNavigation($menu, $level = 0, $limit = 0, $colBrakes = [])
454454
}
455455

456456
$id = $this->getJsId($menuItem->getId());
457-
$output .= '<li ' . $this->getUiId(
458-
$menuItem->getId()
459-
) . ' class="item-' . $itemClass . ' ' . $this->_renderItemCssClass(
460-
$menuItem,
461-
$level
462-
) . ($level == 0 ? '" id="' . $id . '" aria-haspopup="true' : '')
463-
. '" role="menu-item">' . $this->_renderAnchor(
464-
$menuItem,
465-
$level
466-
) . $this->_addSubMenu(
467-
$menuItem,
468-
$level,
469-
$limit,
470-
$id
471-
) . '</li>';
457+
if (count($menu) > 1 || $level != 1) {
458+
$output .= '<li ' . $this->getUiId(
459+
$menuItem->getId()
460+
) . ' class="item-' . $itemClass . ' ' . $this->_renderItemCssClass(
461+
$menuItem,
462+
$level
463+
) . ($level == 0 ? '" id="' . $id . '" aria-haspopup="true' : '')
464+
. '" role="menu-item">' . $this->_renderAnchor(
465+
$menuItem,
466+
$level
467+
) . $this->_addSubMenu(
468+
$menuItem,
469+
$level,
470+
$limit,
471+
$id
472+
) . '</li>';
473+
} else {
474+
$output .= $this->_addSubMenu(
475+
$menuItem,
476+
$level,
477+
$limit,
478+
$id);
479+
}
480+
472481
$itemPosition++;
473482
}
474483

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
<ul id="nav" role="menubar" ><li data-ui-id="magento-backend-system" class="item-system parent last level-0" id="magento-backend-system" aria-haspopup="true" role="menu-item"><a href="#" onclick="return false;" class=""><span>System</span></a><div class="submenu" aria-labelledby="magento-backend-system"><ul role="menu" ><li data-ui-id="magento-backend-system-report" class="item-system-report parent level-1" role="menu-item"><strong class="submenu-group-title" role="presentation"><span>Report</span></strong><div class="submenu"><ul role="menu" ><li data-ui-id="magento-backend-system-report-private-sales" class="item-system-report-private-sales level-2" role="menu-item"><a href="#" onclick="return false;" class=""><span>Private Sales</span></a></li><li data-ui-id="magento-backend-system-report-magento-invite-general" class="item-system-report-magento-invite-general level-2" role="menu-item"><a href="#" onclick="return false;" class=""><span>Invite</span></a></li><li data-ui-id="magento-backend-system-report-magento-invite-customer" class="item-system-report-magento-invite-customer level-2" role="menu-item"><a href="#" onclick="return false;" class=""><span>Invited Customers</span></a></li></ul></div></li></ul></div></li></ul>
1+
<ul id="nav" role="menubar" ><li data-ui-id="magento-backend-system" class="item-system parent last level-0" id="magento-backend-system" aria-haspopup="true" role="menu-item"><a href="#" onclick="return false;" class=""><span>System</span></a><div class="submenu" aria-labelledby="magento-backend-system"><ul role="menu" ><div class="submenu"><ul role="menu" ><li data-ui-id="magento-backend-system-report-private-sales" class="item-system-report-private-sales level-2" role="menu-item"><a href="#" onclick="return false;" class=""><span>Private Sales</span></a></li><li data-ui-id="magento-backend-system-report-magento-invite-general" class="item-system-report-magento-invite-general level-2" role="menu-item"><a href="#" onclick="return false;" class=""><span>Invite</span></a></li><li data-ui-id="magento-backend-system-report-magento-invite-customer" class="item-system-report-magento-invite-customer level-2" role="menu-item"><a href="#" onclick="return false;" class=""><span>Invited Customers</span></a></li></ul></div></ul></div></li></ul>

0 commit comments

Comments
 (0)