From 9abd19e0924c8c3ba7dab7daf5a3074491c53570 Mon Sep 17 00:00:00 2001
From: Chris Hill-Scott <me@quis.cc>
Date: Wed, 22 Jun 2016 09:28:40 +0100
Subject: [PATCH] =?UTF-8?q?Add=20a=20=E2=80=98dashboard=E2=80=99=20link=20?=
 =?UTF-8?q?to=20the=20navigation?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

It’s been unclear that:

- the dashboard exists
- that you click the name of the service to get back to it

So this commit:

- takes the link off the service name
- adds a link labelled ‘Dashboard’ to the navigation
---
 app/assets/stylesheets/components/navigation.scss | 13 ++++---------
 app/templates/main_nav.html                       |  3 ++-
 2 files changed, 6 insertions(+), 10 deletions(-)

diff --git a/app/assets/stylesheets/components/navigation.scss b/app/assets/stylesheets/components/navigation.scss
index 232c334c8..a7cb0d10f 100644
--- a/app/assets/stylesheets/components/navigation.scss
+++ b/app/assets/stylesheets/components/navigation.scss
@@ -2,20 +2,15 @@
 
   padding: $gutter-two-thirds $gutter-half 0 0;
 
-  ul,
-  h2 {
+  h2,
+  li {
     @include core-19;
-    margin: 10px 20px 15px 0;
+    margin: 10px 20px 0 0;
     list-style-type: none;
   }
 
   h2 {
-    font-weight: bold;
-    margin-bottom: 10px;
-  }
-
-  li {
-    margin: 10px 0 0 0;
+    @include bold-19;
   }
 
   a {
diff --git a/app/templates/main_nav.html b/app/templates/main_nav.html
index 8996b9ed7..733379e1d 100644
--- a/app/templates/main_nav.html
+++ b/app/templates/main_nav.html
@@ -46,9 +46,10 @@
 {% else %}
 <nav class="navigation">
   <h2 class="navigation-service-name">
-    <a href="{{ url_for('.service_dashboard', service_id=current_service.id) }}">{{ current_service.name }}</a>
+    {{ 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) %}
     <li><a href="{{ url_for('.choose_template', service_id=current_service.id, template_type='email') }}">Email templates</a></li>
     <li><a href="{{ url_for('.choose_template', service_id=current_service.id, template_type='sms') }}">Text message templates</a></li>