Read only fix and slight header UI tweak

Author: jmgasper

src/components/ChallengeEditor/ChallengeViewTabs/index.js

@@ -96,9 +96,9 @@ const ChallengeViewTabs = ({
   // OR if this isn't a non-self-service draft, permit launching if:
   // a) the current user is either the self-service copilot or is an admin AND
   // b) the challenge is approved
-  const canLaunch = hasBillingAccount && !isReadOnly &&
+  const canLaunch = enableEdit && hasBillingAccount && !isReadOnly &&
     ((!isSelfService && isDraft) ||
-      ((isSelfServiceCopilot || enableEdit || isAdmin) &&
+      ((isSelfServiceCopilot || isAdmin) &&
         challenge.status.toUpperCase() === CHALLENGE_STATUS.APPROVED))
 
   return (

src/components/Tab/Tab.module.scss

@@ -62,6 +62,7 @@
   background-color: #eaf6fd;
   border-radius: 4px 4px 0 0;
   border-bottom: 1px solid #d4d4d4;
+  position: relative;
 
   .item {
     // @include barlow;
@@ -94,7 +95,7 @@
       z-index: 100;
       display: block;
       position: absolute;
-      top: 204px;
+      top: 31px;
     }
   }
 }

src/components/Tab/index.js

@@ -77,8 +77,6 @@ const Tab = ({
 
   return (
     <div className={styles.tabs}>
-      <h1>Work Manager</h1>
-      <hr />
       {tabComponent}
     </div>
   )