gaspersic/freeCodeCamp
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

fix: remove theme nav item (#37028)

Browse Source
This commit is contained in:
Ahmad Abdolsaheb
2019-10-04 19:31:29 +03:00
committed by mrugesh
parent 1130a1c0df
commit ece3dbac80
2 changed files with 4 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
client/src/components/Header/Header.test.js
View File

@ -23,16 +23,16 @@ describe('<NavLinks />', () => {
return acc; return acc;
}, []); }, []);
const expectedLinks = ['/learn', '/', '/portfolio']; const expectedLinks = ['/learn', '/portfolio'];
it('renders to the DOM', () => { it('renders to the DOM', () => {
expect(root).toBeTruthy(); expect(root).toBeTruthy();
}); });
it('has 3 links', () => { it('has 2 links', () => {
expect(aTags.length === 3).toBeTruthy(); expect(aTags.length === 2).toBeTruthy();
}); });
it('has links to learn, main, and portfolio', () => { it('has links to learn and portfolio', () => {
// checks if all links in expected links exist in links // checks if all links in expected links exist in links
expect(expectedLinks.every(elem => links.indexOf(elem) > -1)).toBeTruthy(); expect(expectedLinks.every(elem => links.indexOf(elem) > -1)).toBeTruthy();
}); });

3
client/src/components/Header/components/NavLinks.js
View File

@ -14,9 +14,6 @@ function NavLinks({ displayMenu }) {
className={'nav-list' + (displayMenu ? ' display-flex' : '')} className={'nav-list' + (displayMenu ? ' display-flex' : '')}
role='menu' role='menu'
> >
<li className='nav-theme' role='menuitem'>
<Link to='/'>Light</Link>
</li>
<li className='nav-projects' role='menuitem'> <li className='nav-projects' role='menuitem'>
<Link to='/learn'>Projects</Link> <Link to='/learn'>Projects</Link>
</li> </li>