fix(app-navigation-main-mobile): second level if no children (#1612)

telekom · Feb 23, 2023 · 88e0cd2 · 88e0cd2
1 parent 71dca47
commit 88e0cd2
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/...mponents/src/components/telekom/app-navigation-main-mobile/app-navigation-main-mobile.tsx b/...mponents/src/components/telekom/app-navigation-main-mobile/app-navigation-main-mobile.tsx
@@ -21,6 +21,9 @@ import {
 import { MenuItem } from '../app-interfaces';
 import { findSelected, findRootNode } from '../../../utils/menu-utils';
 
+const hasChildren = (item: any) =>
+  Array.isArray(item.children) && item.children.length > 0;
+
 @Component({
   tag: 'app-navigation-main-mobile',
   styleUrl: 'app-navigation-main-mobile.css',
@@ -77,7 +80,7 @@ export class MainNavigationMobile {
       item.onClick(event);
     }
 
-    if (!selected.children) {
+    if (!hasChildren(selected)) {
       this.closeMenuHandler();
     }
   }
@@ -202,7 +205,7 @@ export class MainNavigationMobile {
             >
               <a
                 aria-current={isActive(item.id) ? 'true' : 'false'}
-                aria-haspopup={item.children ? 'true' : 'false'}
+                aria-haspopup={hasChildren(item) ? 'true' : 'false'}
                 class={`main-navigation-mobile__item-link${
                   isActive(item.id)
                     ? ' main-navigation-mobile__item-link--selected'
@@ -234,7 +237,7 @@ export class MainNavigationMobile {
                 <div class="main-navigation-mobile__item-wrapper">
                   <span>{item.name}</span>
                   {isActive(item.id) && <span class="sr-only">active</span>}
-                  {item.children && (
+                  {hasChildren(item) && (
                     <scale-icon-navigation-right></scale-icon-navigation-right>
                   )}
                 </div>