Merge pull request #934 from maxceem/issue-933

fix: disable "launch active" when no legacy

Author: vikasrohit

src/components/Buttons/PrimaryButton/PrimaryButton.module.scss

@@ -14,6 +14,18 @@
   border: none;
   padding: 0 20px;
 
+  // don't underline if button is done as a link
+  &:hover {
+    text-decoration: none;
+  }
+
+  // hide outline for buttons and links
+  &:focus,
+  &:active,
+  &:visited {
+    outline: none;
+  }
+
   span {
     color: $white;
   }

src/components/ChallengeEditor/index.js

@@ -44,6 +44,7 @@ import AlertModal from '../Modal/AlertModal'
 import PhaseInput from '../PhaseInput'
 import LegacyLinks from '../LegacyLinks'
 import AssignedMemberField from './AssignedMember-Field'
+import Tooltip from '../Tooltip'
 
 const theme = {
   container: styles.modalContainer
@@ -1193,9 +1194,18 @@ class ChallengeEditor extends Component {
               <div className={styles.button}>
                 <PrimaryButton text={'Save Draft'} type={'info'} onClick={this.createDraftHandler} />
               </div>
-              { isDraft && <div className={styles.button}>
-                <PrimaryButton text={'Launch as Active'} type={'info'} onClick={this.toggleLaunch} />
-              </div>}
+              {isDraft && (
+                <div className={styles.button}>
+                  {challenge.legacyId ? (
+                    <PrimaryButton text={'Launch as Active'} type={'info'} onClick={this.toggleLaunch} />
+                  ) : (
+                    <Tooltip content='Legacy project is not yet created'>
+                      {/* Don't disable button for real inside tooltip, otherwise mouseEnter/Leave events work not good */}
+                      <PrimaryButton text={'Launch as Active'} type={'disabled'} />
+                    </Tooltip>
+                  )}
+                </div>
+              )}
             </div>}
             {!isLoading && isActive && <div className={styles.buttonContainer}>
               {/* <div className={styles.button}>

src/components/LegacyLinks/LegacyLinks.module.scss

@@ -13,19 +13,4 @@
     outline: none;
     white-space: nowrap;
   }
-
-  a {
-    &:hover {
-      text-decoration: none;
-    }
-    &:focus,
-    &:active,
-    &:visited {
-      outline: none;
-    }
-
-    button:focus {
-      outline: none;
-    }
-  }
 }