Merge pull request #125 from gets0ul/issue_104

Fix for Issue #104 - Logo Link

Author: luizrrodrigues

src/components/TopNav/MobileNav.js

@@ -6,7 +6,7 @@ import IconMenu from '../../assets/images/icon-menu.svg'
 import IconMagnifyingGlass from '../../assets/images/magnifying_glass.svg'
 import { config } from 'topcoder-react-utils'
 
-const MobileNav = ({ showLeftMenu, onClickLeftMenu, logo, rightMenu }) => (<div>
+const MobileNav = ({ showLeftMenu, onClickLeftMenu, logo, onClickLogo, rightMenu }) => (<div>
   <div className={styles.mobileNav}>
     <div className={styles.leftMenuContainer}>
       <button className={styles.menuBtn} onClick={onClickLeftMenu}>
@@ -17,7 +17,9 @@ const MobileNav = ({ showLeftMenu, onClickLeftMenu, logo, rightMenu }) => (<div>
         )}
       </button>
     </div>
-    <a href='/'>
+    <a
+      onClick={(e) => onClickLogo(e)}
+      href='/'>
       {logo}
     </a>
     {rightMenu && (
@@ -49,6 +51,7 @@ MobileNav.propTypes = {
   showLeftMenu: PropTypes.bool,
   onClickLeftMenu: PropTypes.func,
   logo: PropTypes.node,
+  onClickLogo: PropTypes.func,
   rightMenu: PropTypes.node
 }
 

src/components/TopNav/PrimaryNav.js

@@ -45,7 +45,7 @@ const PrimaryNav = ({
         <div className={styles.primaryNav} ref={createSetRef('primaryNav')}>
           <a
             className={cn(styles.tcLogo, collapsed && styles.tcLogoPush)}
-            onClick={onClickLogo}
+            onClick={(e) => onClickLogo(e)}
             href='/'
           >
             {logo}

src/components/TopNav/index.js

@@ -2,6 +2,7 @@ import React, { useState, useMemo, useEffect, useLayoutEffect, useCallback } fro
 import PropTypes from 'prop-types'
 import cn from 'classnames'
 import _ from 'lodash'
+import { config } from 'topcoder-react-utils'
 
 import styles from './index.module.scss'
 
@@ -179,8 +180,9 @@ const TopNav = ({
     }, 0)
   }
 
-  const handleClickLogo = () => {
-
+  const handleClickLogo = (e) => {
+    e.preventDefault()
+    window.location = loggedIn ? config.URL.HOME : config.URL.BASE
   }
 
   const expandMenu = (menuId, menu2Id) => {
@@ -484,6 +486,7 @@ const TopNav = ({
         <MobileNav
           showLeftMenu={showLeftMenu}
           logo={logo}
+          onClickLogo={handleClickLogo}
           rightMenu={rightMenu}
           onClickLeftMenu={handleClickLeftMenu}
         />