Skip to content

Commit 8b8f746

Browse files
committed
hide white secondary menus when empty
1 parent e4252cd commit 8b8f746

File tree

1 file changed

+24
-21
lines changed

1 file changed

+24
-21
lines changed

src/components/TopNav/SubNav.js

Lines changed: 24 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -16,27 +16,30 @@ const SubNav = ({
1616
indicatorX,
1717
createHandleClickItem,
1818
createSetRef
19-
}) => (
20-
<div className={cn(styles.secondaryNav, open && styles.secondaryNavOpen)}>
21-
<div className={styles.secondaryNavLinkContainer}>
22-
{menu && (isSecondaryMenu ? menu.secondaryMenu : menu.subMenu) && (isSecondaryMenu ? menu.secondaryMenu : menu.subMenu).map((level3, i) => {
23-
const to = _.isEmpty(level3.link) ? level3.href : level3.link
24-
return (
25-
<Link
26-
className={cn(styles.secondaryNavItem, level3.id === activeChildId && styles.secondaryNavItemOpen)}
27-
key={`level3-${i}`}
28-
to={to}
29-
onClick={!level3.openNewTab && createHandleClickItem(level3.id)}
30-
openNewTab={level3.openNewTab}
31-
>
32-
<span ref={createSetRef(level3.id)}>{level3.title}</span>
33-
<span className={cn(styles.indicator)} />
34-
</Link>
35-
)
36-
})}
37-
<IconSelect isResize={isResize} show={showIndicator} x={indicatorX} /> </div>
38-
</div>
39-
)
19+
}) => {
20+
if (!menu || (isSecondaryMenu ? menu.secondaryMenu : menu.subMenu).length === 0) return null
21+
return (
22+
<div className={cn(styles.secondaryNav, open && styles.secondaryNavOpen)}>
23+
<div className={styles.secondaryNavLinkContainer}>
24+
{menu && (isSecondaryMenu ? menu.secondaryMenu : menu.subMenu) && (isSecondaryMenu ? menu.secondaryMenu : menu.subMenu).map((level3, i) => {
25+
const to = _.isEmpty(level3.link) ? level3.href : level3.link
26+
return (
27+
<Link
28+
className={cn(styles.secondaryNavItem, level3.id === activeChildId && styles.secondaryNavItemOpen)}
29+
key={`level3-${i}`}
30+
to={to}
31+
onClick={!level3.openNewTab && createHandleClickItem(level3.id)}
32+
openNewTab={level3.openNewTab}
33+
>
34+
<span ref={createSetRef(level3.id)}>{level3.title}</span>
35+
<span className={cn(styles.indicator)} />
36+
</Link>
37+
)
38+
})}
39+
<IconSelect isResize={isResize} show={showIndicator} x={indicatorX} /> </div>
40+
</div>
41+
)
42+
}
4043

4144
SubNav.propTypes = {
4245
open: PropTypes.bool,

0 commit comments

Comments
 (0)