Skip to content

Commit 55876e6

Browse files
committed
dynamically set the header link colors
1 parent 4fa9419 commit 55876e6

File tree

2 files changed

+8
-5
lines changed

2 files changed

+8
-5
lines changed

src/compounds/Header/Header.jsx

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3,11 +3,11 @@ import PropTypes from 'prop-types'
33
import { Container, Nav, Navbar } from 'react-bootstrap'
44
import Logo from '../Logo/Logo'
55

6-
const Header = ({ auth, logo, navLinks, userSession }) => {
6+
const Header = ({ auth, linkColor, logo, navLinks, userSession }) => {
77
const { src, alt } = logo
88

99
return (
10-
<Navbar bg='secondary' expand='lg'>
10+
<Navbar bg='primary' expand='lg'>
1111
<Container>
1212
<Navbar.Brand className='w-50'>
1313
<Logo src={src} alt={alt} height={45} />
@@ -17,7 +17,7 @@ const Header = ({ auth, logo, navLinks, userSession }) => {
1717
<Nav className='ms-auto'>
1818
{navLinks.map((nav) => (
1919
<Nav.Link
20-
className='link-dark'
20+
className={`link-${linkColor}`}
2121
href={nav.path}
2222
key={`${nav.label}-nav-link`}
2323
onClick={nav.onClick}
@@ -26,9 +26,9 @@ const Header = ({ auth, logo, navLinks, userSession }) => {
2626
</Nav.Link>
2727
))}
2828
{userSession ? (
29-
<Nav.Link className='link-dark' onClick={auth.signOut}>Sign Out</Nav.Link>
29+
<Nav.Link className={`link-${linkColor}`} onClick={auth.signOut}>Sign Out</Nav.Link>
3030
) : (
31-
<Nav.Link className='link-dark' onClick={auth.signIn}>Sign In</Nav.Link>
31+
<Nav.Link className={`link-${linkColor}`} onClick={auth.signIn}>Sign In</Nav.Link>
3232
)}
3333
</Nav>
3434
</Navbar.Collapse>
@@ -42,6 +42,7 @@ Header.propTypes = {
4242
signIn: PropTypes.func.isRequired,
4343
signOut: PropTypes.func.isRequired,
4444
}).isRequired,
45+
linkColor: PropTypes.string,
4546
logo: PropTypes.shape({
4647
src: PropTypes.string.isRequired,
4748
alt: PropTypes.string,
@@ -55,6 +56,7 @@ Header.propTypes = {
5556
}
5657

5758
Header.defaultProps = {
59+
linkColor: 'dark',
5860
userSession: null,
5961
}
6062

src/compounds/Header/Header.stories.jsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,7 @@ LoggedOut.args = {
3535
signIn: () => console.log('I am signing in!'),
3636
signOut: () => console.log('I am signing out!'),
3737
},
38+
linkColor: 'light',
3839
logo: {
3940
src: logoFull,
4041
alt: 'The company logo',

0 commit comments

Comments
 (0)