From 0b0b1a653d321ce9db8410ce04b9514419a103be Mon Sep 17 00:00:00 2001
From: Robin van Zanten <38441984+RCVZ@users.noreply.github.com>
Date: Tue, 4 May 2021 21:01:37 +0200
Subject: [PATCH] feat(playlist): add playlist navigation to header

---
 src/components/Header/Header.test.tsx         |  9 ++++++-
 src/components/Header/Header.tsx              | 10 ++++---
 .../Header/__snapshots__/Header.test.tsx.snap | 16 +++---------
 src/components/Layout/Layout.tsx              | 25 +++++++++++++++---
 .../Layout/__snapshots__/Layout.test.tsx.snap | 26 -------------------
 5 files changed, 40 insertions(+), 46 deletions(-)

diff --git a/src/components/Header/Header.test.tsx b/src/components/Header/Header.test.tsx
index 2317c36ae..d0b851100 100644
--- a/src/components/Header/Header.test.tsx
+++ b/src/components/Header/Header.test.tsx
@@ -1,12 +1,19 @@
 import React from 'react';
 
 import { render } from '../../testUtils';
+import ButtonLink from '../ButtonLink/ButtonLink';
 
 import Header from './Header';
 
 describe('<Header />', () => {
   test('renders header', () => {
-    const { container } = render(<Header onMenuButtonClick={jest.fn()} />);
+    const playlistMenuItems = [<ButtonLink key="key" label="Home" to="/" />];
+    const { container } = render(
+      <Header
+        onMenuButtonClick={jest.fn()}
+        playlistMenuItems={playlistMenuItems}
+      />,
+    );
 
     expect(container).toMatchSnapshot();
   });
diff --git a/src/components/Header/Header.tsx b/src/components/Header/Header.tsx
index 34aa41e46..f1c5a1ea8 100644
--- a/src/components/Header/Header.tsx
+++ b/src/components/Header/Header.tsx
@@ -12,11 +12,15 @@ type TypeHeader = 'static' | 'fixed';
 type Props = {
   headerType?: TypeHeader;
   onMenuButtonClick: () => void;
+  playlistMenuItems: JSX.Element[];
+  logoSrc?: string;
 };
 
 const Header: React.FC<Props> = ({
   headerType = 'static',
   onMenuButtonClick,
+  playlistMenuItems,
+  logoSrc,
 }) => {
   return (
     <header className={classNames(styles.header, styles[headerType])}>
@@ -29,13 +33,11 @@ const Header: React.FC<Props> = ({
         >
           <Menu />
         </div>
-        <Logo src="https://cdn.jwplayer.com/images/HXyBCU5N.png" />
+        {logoSrc && <Logo src={logoSrc} />}
         <nav className={styles.nav} aria-label="menu">
           <ButtonLink label="Home" to="/" />
-          {/* mock */}
-          <ButtonLink label="Playlist" to="/p/:id" />
+          {playlistMenuItems}
           <ButtonLink label="Settings" to="/u" />
-          {/* mock */}
         </nav>
         <div className={styles.search}></div>
       </div>
diff --git a/src/components/Header/__snapshots__/Header.test.tsx.snap b/src/components/Header/__snapshots__/Header.test.tsx.snap
index e2f9ea16f..ab41a939b 100644
--- a/src/components/Header/__snapshots__/Header.test.tsx.snap
+++ b/src/components/Header/__snapshots__/Header.test.tsx.snap
@@ -31,15 +31,6 @@ exports[`<Header /> renders header 1`] = `
           </g>
         </svg>
       </div>
-      <div
-        class="brand"
-      >
-        <img
-          alt="logo"
-          class="logo"
-          src="https://cdn.jwplayer.com/images/HXyBCU5N.png"
-        />
-      </div>
       <nav
         aria-label="menu"
         class="nav"
@@ -54,11 +45,12 @@ exports[`<Header /> renders header 1`] = `
           </span>
         </a>
         <a
-          class="link"
-          href="/p/:id"
+          aria-current="page"
+          class="link active"
+          href="/"
         >
           <span>
-            Playlist
+            Home
           </span>
         </a>
         <a
diff --git a/src/components/Layout/Layout.tsx b/src/components/Layout/Layout.tsx
index d4265ef61..69c10d94c 100644
--- a/src/components/Layout/Layout.tsx
+++ b/src/components/Layout/Layout.tsx
@@ -1,5 +1,7 @@
-import React, { ReactNode, FC, useState } from 'react';
+import React, { ReactNode, FC, useState, useContext } from 'react';
 
+import ButtonLink from '../ButtonLink/ButtonLink';
+import { ConfigContext } from '../../providers/configProvider';
 import Header from '../Header/Header';
 import SideBar from '../SideBar/SideBar';
 
@@ -10,12 +12,29 @@ type LayoutProps = {
 };
 
 const Layout: FC<LayoutProps> = ({ children }) => {
+  const { menu, assets } = useContext(ConfigContext);
   const [sideBarOpen, setSideBarOpen] = useState(false);
 
+  const playlistMenuItems = menu.map((item) => (
+    <ButtonLink
+      key={item.playlistId}
+      label={item.label}
+      to={`/p/${item.playlistId}`}
+    />
+  ));
+
   return (
     <div className={styles.layout}>
-      <Header onMenuButtonClick={() => setSideBarOpen(true)} />
-      <SideBar isOpen={sideBarOpen} onClose={() => setSideBarOpen(false)} />
+      <Header
+        onMenuButtonClick={() => setSideBarOpen(true)}
+        playlistMenuItems={playlistMenuItems}
+        logoSrc={assets.banner}
+      />
+      <SideBar
+        isOpen={sideBarOpen}
+        onClose={() => setSideBarOpen(false)}
+        playlistMenuItems={playlistMenuItems}
+      />
       {children}
     </div>
   );
diff --git a/src/components/Layout/__snapshots__/Layout.test.tsx.snap b/src/components/Layout/__snapshots__/Layout.test.tsx.snap
index 5b2a3d8e9..2f7c48120 100644
--- a/src/components/Layout/__snapshots__/Layout.test.tsx.snap
+++ b/src/components/Layout/__snapshots__/Layout.test.tsx.snap
@@ -32,15 +32,6 @@ exports[`<Layout /> renders layout 1`] = `
             </g>
           </svg>
         </div>
-        <div
-          class="brand"
-        >
-          <img
-            alt="logo"
-            class="logo"
-            src="https://cdn.jwplayer.com/images/HXyBCU5N.png"
-          />
-        </div>
         <nav
           aria-label="menu"
           class="nav"
@@ -54,14 +45,6 @@ exports[`<Layout /> renders layout 1`] = `
               Home
             </span>
           </a>
-          <a
-            class="link"
-            href="/p/:id"
-          >
-            <span>
-              Playlist
-            </span>
-          </a>
           <a
             class="link"
             href="/u"
@@ -117,15 +100,6 @@ exports[`<Layout /> renders layout 1`] = `
             Home
           </span>
         </a>
-        <a
-          aria-current="page"
-          class="link active"
-          href="/"
-        >
-          <span>
-            Test
-          </span>
-        </a>
         <hr
           class="divider"
         />