Merge pull request #91 from nikolabjelo/master

Navbar : adapting icon size, making homepage use 'Link' instead of 'a', making external links open a new tab

client/src/styles/styles.scss

@@ -235,6 +235,9 @@ header.menu{
               &:hover{
                 color: $advanced-algos-red;
               }
+              svg{
+                padding-top: 10px;
+              }
             }
             ul.subMenu{
               display: none;
@@ -293,6 +296,9 @@ header.menu{
               &.selected{
                 color: $advanced-algos-red;
               }
+              svg{
+                padding-top: 10px;
+              }
             }
             ul.subMenu{
               display: none;

client/src/views/layout/header/index.js

@@ -98,7 +98,7 @@ class Header extends Component {
           <ul className='subMenu'>
             { bigScreen
               ? ''
-              : <li key={index + 'home'}><a href={to}> <Icon /> Homepage </a></li>
+              : <li key={index + 'home'}><Link to={to}> <Icon /> Homepage </Link></li>
             }
             {
               submenus.map(({ icon: SubIcon, to: subTo, title: subTitle, externalLink, authenticated: subAuthenticated }, subindex) => {
@@ -107,7 +107,7 @@ class Header extends Component {
                 }
                 if (externalLink) {
                   return (
-                    <li key={subindex}><a href={subTo}> <SubIcon /> {subTitle} </a></li>
+                    <li key={subindex}><a href={subTo} target='_blank'> <SubIcon /> {subTitle} </a></li>
                   )
                 }
                 return (