From d95b46dd196c8aa3b37c2424c23ff8740ad64a70 Mon Sep 17 00:00:00 2001
From: Brian Talbot <btalbot@edx.org>
Date: Mon, 25 Nov 2013 19:59:59 -0500
Subject: [PATCH] Studio: adding back in links (and supporting styling) to ToS
 and Privacy Policy to footer and sign up UI STUD-151

---
 cms/envs/common.py                    |  9 +--------
 cms/static/sass/_variables.scss       |  9 +++++++++
 cms/static/sass/elements/_footer.scss | 21 ++++-----------------
 cms/templates/signup.html             |  4 +++-
 cms/templates/widgets/footer.html     | 14 +++++++-------
 5 files changed, 24 insertions(+), 33 deletions(-)

diff --git a/cms/envs/common.py b/cms/envs/common.py
index a8c293b57f..0c9166a7f9 100644
--- a/cms/envs/common.py
+++ b/cms/envs/common.py
@@ -414,14 +414,7 @@ INSTALLED_APPS = (
 EDXMKTG_COOKIE_NAME = 'edxloggedin'
 MKTG_URLS = {}
 MKTG_URL_LINK_MAP = {
-    'ABOUT': 'about_edx',
-    'CONTACT': 'contact',
-    'FAQ': 'help_edx',
-    'COURSES': 'courses',
-    'ROOT': 'root',
-    'TOS': 'tos',
-    'HONOR': 'honor',
-    'PRIVACY': 'privacy_edx',
+
 }
 
 COURSES_WITH_UNSAFE_CODE = []
diff --git a/cms/static/sass/_variables.scss b/cms/static/sass/_variables.scss
index 093d3a957a..ecb63e1b85 100644
--- a/cms/static/sass/_variables.scss
+++ b/cms/static/sass/_variables.scss
@@ -172,6 +172,15 @@ $tmg-f3: 0.125s;
 
 // ====================
 
+// archetype UI
+$ui-action-primary-color: $blue-u2;
+$ui-action-primary-color-focus: $blue-s1;
+
+$ui-link-color: $blue-u2;
+$ui-link-color-focus: $blue-s1;
+
+// ====================
+
 // specific UI
 $ui-notification-height: ($baseline*10);
 $ui-update-color: $blue-l4;
diff --git a/cms/static/sass/elements/_footer.scss b/cms/static/sass/elements/_footer.scss
index 94085ea7c1..268afc1a2e 100644
--- a/cms/static/sass/elements/_footer.scss
+++ b/cms/static/sass/elements/_footer.scss
@@ -23,10 +23,10 @@
     }
 
     a {
-      color: $gray;
+      color: $ui-link-color;
 
       &:hover, &:active {
-        color: $gray-d2;
+        color: $ui-link-color-focus;
       }
     }
 
@@ -37,7 +37,7 @@
 
       .nav-item {
         display: inline-block;
-        margin-right: ($baseline/2);
+        margin-right: ($baseline/4);
 
         &:last-child {
           margin-right: 0;
@@ -45,7 +45,7 @@
 
         a {
           border-radius: 2px;
-          padding: ($baseline/2) ($baseline*0.75);
+          padding: ($baseline/2) ($baseline/2);
           background: transparent;
 
           [class^="icon-"] {
@@ -54,19 +54,6 @@
             display: inline-block;
             vertical-align: middle;
             margin-right: ($baseline/4);
-            color: $gray-l1;
-          }
-
-          &:hover, &:active {
-            color: $gray-d2;
-
-            [class^="icon-"] {
-              color: $gray-d2;
-            }
-          }
-
-          &.is-active {
-            color: $gray-d2;
           }
         }
       }
diff --git a/cms/templates/signup.html b/cms/templates/signup.html
index d9a733ae47..422c152b50 100644
--- a/cms/templates/signup.html
+++ b/cms/templates/signup.html
@@ -60,7 +60,9 @@
 
             <li class="field checkbox required" id="field-tos">
               <input id="tos" name="terms_of_service" type="checkbox" value="true" />
-              <label for="tos">${_("I agree to the Terms of Service")}</label>
+              <label for="tos">
+                ${_("I agree to the {a_start} Terms of Service {a_end}").format(a_start='<a data-rel="edx.org" href="{}">'.format(marketing_link('TOS')), a_end="</a>")}
+              </label>
             </li>
           </ol>
         </fieldset>
diff --git a/cms/templates/widgets/footer.html b/cms/templates/widgets/footer.html
index db7d5fb3f8..bd7d0c6073 100644
--- a/cms/templates/widgets/footer.html
+++ b/cms/templates/widgets/footer.html
@@ -1,25 +1,25 @@
 <%! from django.core.urlresolvers import reverse %>
 <%! from django.utils.translation import ugettext as _ %>
 
-<div class="wrapper-footer wrapper">  
+<div class="wrapper-footer wrapper">
   <footer class="primary" role="contentinfo">
     <div class="colophon">
       <p>&copy; 2013 <a href="http://www.edx.org" rel="external">edX</a>. ${ _("All rights reserved.")}</p>
     </div>
-    
+
     <nav class="nav-peripheral">
       <ol>
-        <!-- <li class="nav-item nav-peripheral-tos">
-          <a href="#">Terms of Service</a>
+        <li class="nav-item nav-peripheral-tos">
+          <a data-rel="edx.org" href="${marketing_link('TOS')}">${_("Terms of Service")}</a>
         </li>
         <li class="nav-item nav-peripheral-pp">
-          <a href="#">Privacy Policy</a>
-        </li> -->
+          <a data-rel="edx.org" href="${marketing_link('PRIVACY')}">${_("Privacy Policy")}</a>
+        </li>
         % if user.is_authenticated():
         <li class="nav-item nav-peripheral-feedback">
           <a href="http://help.edge.edx.org/discussion/new" class="show-tender" title="${_('Use our feedback tool, Tender, to share your feedback')}">${_("Contact Us")}</a>
         </li>
-        % endif    
+        % endif
       </ol>
     </nav>
   </footer>