Merge pull request #16368 from code-dot-org/minimal-ui-tips-for-stude…

…nt-homepage Student homepage: only one UI tip should show
code-dot-org · Jul 13, 2017 · 3051bbd · 3051bbd
2 parents 87452d5 + a1b4766
commit 3051bbd
Showing 1 changed file with 34 additions and 4 deletions.
diff --git a/apps/src/sites/studio/pages/home/_homepage.js b/apps/src/sites/studio/pages/home/_homepage.js
@@ -20,7 +20,7 @@ function showHomepage() {
 
   ReactDOM.render (
     <div>
-      {showUiTips && (
+      {(isTeacher && showUiTips) && (
         <UiTips
           userId={userId}
           tipId="homepage_header"
@@ -40,9 +40,39 @@ function showHomepage() {
           }}
           tips={
             [
-              {type: "initial", position: {top: 80, left: 100}, text: i18n.homepageUiTipKeyLinks(), arrowDirection: "up"},
-              {type: "initial", position: {top: 80, right: 15}, text: i18n.homepageUiTipOtherLinks(), arrowDirection: "up_corner"},
-              {type: "triggered", position: {top: 80, right: 15}, text: i18n.homepageUiTipAlreadyHome(), triggerId: "logo_home_link", arrowDirection: "up_corner"}
+              {
+                type: "initial",
+                position: {top: 80, left: 100},
+                text: i18n.homepageUiTipKeyLinks(),
+                arrowDirection: "up"
+              },
+              {
+                type: "initial",
+                position: {top: 80, right: 15},
+                text: i18n.homepageUiTipOtherLinks(),
+                arrowDirection: "up_corner"
+              },
+              {
+                type: "triggered",
+                position: {top: 80, right: 15},
+                text: i18n.homepageUiTipAlreadyHome(),
+                triggerId: "logo_home_link",
+                arrowDirection: "up_corner"}
+            ]}
+        />
+      )}
+
+      {(!isTeacher && showUiTips) && (
+        <UiTips
+          tips={
+            [
+              {
+                type: "triggered",
+                position: {top: 80, right: 15},
+                text: i18n.homepageUiTipAlreadyHome(),
+                triggerId: "logo_home_link",
+                arrowDirection: "up_corner"
+              }
             ]}
         />
       )}