From b4ad1ae782765cb70342051d5427476dcc98daa3 Mon Sep 17 00:00:00 2001
From: Axel Peter <axel.peter@corp.ovh.com>
Date: Tue, 27 Feb 2018 10:57:25 +0100
Subject: [PATCH 1/2] :lipstick: style(navbar): hide empty sublinks

---
 packages/oui-navbar/src/index.spec.data.json  | 6 ++++++
 packages/oui-navbar/src/menu/navbar-menu.html | 4 ++--
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/packages/oui-navbar/src/index.spec.data.json b/packages/oui-navbar/src/index.spec.data.json
index 67455274..5a2e85ad 100644
--- a/packages/oui-navbar/src/index.spec.data.json
+++ b/packages/oui-navbar/src/index.spec.data.json
@@ -354,6 +354,12 @@
                             "url": "#!/oui-angular/navbar"
                         }
                     ]
+                },
+                {
+                    "name": "empty",
+                    "label": "Not visible",
+                    "title": "Empty",
+                    "subLinks": []
                 }
             ]
         },
diff --git a/packages/oui-navbar/src/menu/navbar-menu.html b/packages/oui-navbar/src/menu/navbar-menu.html
index cf7d0812..7a3f1d9d 100644
--- a/packages/oui-navbar/src/menu/navbar-menu.html
+++ b/packages/oui-navbar/src/menu/navbar-menu.html
@@ -77,7 +77,7 @@ <h3 class="oui-navbar-menu__title" ng-bind="::$ctrl.headerTitle"></h3>
         <!-- Button for Next Level -->
         <button class="oui-navbar-link" type="button"
             aria-haspopup="true" aria-expanded="{{!!$ctrl.navigation[menuLink.name]}}"
-            ng-if="::!!menuLink.subLinks"
+            ng-if="::!!menuLink.subLinks && !!menuLink.subLinks.length"
             ng-attr-aria-label="{{::!!menuLink.label ? menuLink.label : null}}"
             ng-attr-title="{{::!!menuLink.label ? menuLink.label : null}}"
             ng-class="::{
@@ -93,7 +93,7 @@ <h3 class="oui-navbar-menu__title" ng-bind="::$ctrl.headerTitle"></h3>
 
         <!-- Menu for Next Level -->
         <oui-navbar-menu
-            ng-if="::!!menuLink.subLinks"
+            ng-if="::!!menuLink.subLinks && !!menuLink.subLinks.length"
             ng-class="::!!$ctrl.childrenClass ? $ctrl.childrenClass : null"
             children-class="{{::$ctrl.childrenClass}}"
             back-button="true"

From 1145c405222c61d3a28ba2dbd89cd8c4fd463270 Mon Sep 17 00:00:00 2001
From: Axel Peter <axel.peter@corp.ovh.com>
Date: Tue, 27 Feb 2018 12:53:46 +0100
Subject: [PATCH 2/2] :lipstick: style(navbar): update children menu as mobile
 only

---
 packages/oui-navbar/src/menu/navbar-menu.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/oui-navbar/src/menu/navbar-menu.html b/packages/oui-navbar/src/menu/navbar-menu.html
index 7a3f1d9d..f018f239 100644
--- a/packages/oui-navbar/src/menu/navbar-menu.html
+++ b/packages/oui-navbar/src/menu/navbar-menu.html
@@ -92,7 +92,7 @@ <h3 class="oui-navbar-menu__title" ng-bind="::$ctrl.headerTitle"></h3>
         <!-- Button for Next Level -->
 
         <!-- Menu for Next Level -->
-        <oui-navbar-menu
+        <oui-navbar-menu class="oui-navbar_mobile-only"
             ng-if="::!!menuLink.subLinks && !!menuLink.subLinks.length"
             ng-class="::!!$ctrl.childrenClass ? $ctrl.childrenClass : null"
             children-class="{{::$ctrl.childrenClass}}"