From ee78d0bf596871e0ee11d8fcf3a108486ee4e0fb Mon Sep 17 00:00:00 2001
From: Chris Hill-Scott <me@quis.cc>
Date: Fri, 18 Nov 2016 11:18:29 +0000
Subject: [PATCH] Move switch service out of black header

Also following the pattern work that Tim and Stephen have done.
---
 app/assets/stylesheets/_grids.scss            |  2 +-
 app/assets/stylesheets/app.scss               | 12 ++++----
 .../stylesheets/components/navigation.scss    | 28 +++++++++++++++----
 app/assets/stylesheets/views/dashboard.scss   |  5 ++++
 app/templates/admin_template.html             |  3 --
 app/templates/main_nav.html                   |  3 --
 app/templates/withnav_template.html           |  6 ++++
 7 files changed, 42 insertions(+), 17 deletions(-)

diff --git a/app/assets/stylesheets/_grids.scss b/app/assets/stylesheets/_grids.scss
index cdd68defe..a52facba4 100644
--- a/app/assets/stylesheets/_grids.scss
+++ b/app/assets/stylesheets/_grids.scss
@@ -46,7 +46,7 @@
 .align-with-heading {
   display: block;
   text-align: center;
-  margin-top: $gutter;
+  margin-top: 13px;
   padding-left: 2px;
   padding-right: 2px;
 }
diff --git a/app/assets/stylesheets/app.scss b/app/assets/stylesheets/app.scss
index de08eee3c..0ecc41389 100644
--- a/app/assets/stylesheets/app.scss
+++ b/app/assets/stylesheets/app.scss
@@ -56,14 +56,16 @@ a {
 
 .column-main {
 
-  > .heading-large {
-    margin-top: $gutter;
+  > .heading-large,
+  > .heading-medium {
+    margin: 10px 0 15px 0;
   }
 
-  > .grid-row  {
+  > .grid-row:first-child  {
 
-    .heading-large {
-      margin-top: $gutter;
+    .heading-large,
+    .heading-medium {
+      margin: 10px 0 15px 0;
     }
 
   }
diff --git a/app/assets/stylesheets/components/navigation.scss b/app/assets/stylesheets/components/navigation.scss
index 8b16a157c..451a54910 100644
--- a/app/assets/stylesheets/components/navigation.scss
+++ b/app/assets/stylesheets/components/navigation.scss
@@ -2,17 +2,35 @@
 
   padding: 0 $gutter 0 0;
 
+  &-service {
+
+    padding: 11px 0 9px 0;
+    border-bottom: 1px solid $border-colour;
+    margin: 0 0 10px;
+    position: relative;
+
+    h2 {
+      @include bold-19;
+      margin: 0 0 0 0;
+    }
+
+    &-switch {
+      @include core-16;
+      text-align: right;
+      position: absolute;
+      top: 0;
+      right: 0;
+      padding: 13px 0 9px 15px;
+    }
+
+  }
+
   li {
     @include core-19;
     margin: 10px 0 0 0;
     list-style-type: none;
   }
 
-  h2 {
-    @include bold-19;
-    margin: $gutter 0 0 0;
-  }
-
   a {
 
     &:link,
diff --git a/app/assets/stylesheets/views/dashboard.scss b/app/assets/stylesheets/views/dashboard.scss
index 80f0b994d..9ad0c5843 100644
--- a/app/assets/stylesheets/views/dashboard.scss
+++ b/app/assets/stylesheets/views/dashboard.scss
@@ -11,6 +11,11 @@
       border: 0;
     }
   }
+
+  > .heading-medium {
+    margin-top: 10px;
+  }
+
 }
 
 .keyline-block {
diff --git a/app/templates/admin_template.html b/app/templates/admin_template.html
index 55501e7e8..c0d4dfa4a 100644
--- a/app/templates/admin_template.html
+++ b/app/templates/admin_template.html
@@ -45,9 +45,6 @@
             <li>
               <a href="{{ url_for('main.user_profile') }}">{{ current_user.name }}</a>
             </li>
-            <li>
-              <a href="{{ url_for('main.choose_service') }}">Switch service</a>
-            </li>
             {% if current_user.has_permissions(admin_override=True) %}
                 <li>
                   <a href="{{ url_for('main.platform_admin') }}">Platform admin</a>
diff --git a/app/templates/main_nav.html b/app/templates/main_nav.html
index 0ef4ddc35..7305fb577 100644
--- a/app/templates/main_nav.html
+++ b/app/templates/main_nav.html
@@ -42,9 +42,6 @@
   {% endcall %}
 {% else %}
 <nav class="navigation">
-  <h2 class="navigation-service-name">
-    {{ current_service.name }}
-  </h2>
   <ul>
     <li><a href="{{ url_for('.service_dashboard', service_id=current_service.id) }}">Dashboard</a></li>
   {% if current_user.has_permissions(['view_activity', 'manage_templates', 'manage_api_keys'], admin_override=True, any_=True) %}
diff --git a/app/templates/withnav_template.html b/app/templates/withnav_template.html
index caf0aa651..b72d1f6b7 100644
--- a/app/templates/withnav_template.html
+++ b/app/templates/withnav_template.html
@@ -1,6 +1,12 @@
 {% extends "admin_template.html" %}
 
 {% block fullwidth_content %}
+  <div class="navigation-service">
+    <h2 class="navigation-service-name">
+      {{ current_service.name }}
+    </h2>
+    <a href="{{ url_for('main.choose_service') }}" class="navigation-service-switch">Switch service</a>
+  </div>
   <div class="grid-row">
     <div class="column-one-third">
       {% include "main_nav.html" %}