fix issue #1195 #1203 #1204

Author: yoution

src/components/LegacyLinks/index.js

@@ -5,7 +5,8 @@ import React, { useCallback } from 'react'
 import PropTypes from 'prop-types'
 import cn from 'classnames'
 import styles from './LegacyLinks.module.scss'
-import { ONLINE_REVIEW_URL } from '../../config/constants'
+import Tooltip from '../Tooltip'
+import { MESSAGE, ONLINE_REVIEW_URL } from '../../config/constants'
 
 const LegacyLinks = ({ challenge, challengeView }) => {
   const onClick = useCallback((e) => {
@@ -15,7 +16,12 @@ const LegacyLinks = ({ challenge, challengeView }) => {
   const orUrl = `${ONLINE_REVIEW_URL}/review/actions/ViewProjectDetails?pid=${challenge.legacyId}`
   return (
     <div className={styles.container}>
-      (  <a href={orUrl} target={'_blank'} onClick={onClick}>Online Review</a> )
+      (  {challenge.legacyId ? <a href={orUrl} target={'_blank'} onClick={onClick}>Online Review</a>
+        : <Tooltip content={MESSAGE.NO_LEGACY_CHALLENGE}>
+          {/* Don't disable button for real inside tooltip, otherwise mouseEnter/Leave events work not good */}
+          <a disabled>Online Review</a>
+        </Tooltip>}
+      )
       <div>
         { challengeView && challenge.discussions && challenge.discussions.map(d => (
           <div key={d.id} className={cn(styles.row, styles.topRow)}>