From 86bdd0ad86ab4485863f06155ccf4f0df5b3e84b Mon Sep 17 00:00:00 2001
From: Chris Hill-Scott <me@quis.cc>
Date: Tue, 26 Jan 2016 14:14:33 +0000
Subject: [PATCH 1/3] Show all links in side navigation again

These were temporarily hidden for hack day, to not distract participants.
---
 app/templates/main_nav.html | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/app/templates/main_nav.html b/app/templates/main_nav.html
index 6c7f7fb21..609d6eef5 100644
--- a/app/templates/main_nav.html
+++ b/app/templates/main_nav.html
@@ -3,9 +3,9 @@
     <li><a href="{{ url_for('.service_dashboard', service_id=service_id) }}">Dashboard</a></li>
   </ul>
   <ul>
-    <!--<li><a href="{{ url_for('.send_sms', service_id=service_id) }}">Send text messages</a></li>-->
-    <!--<li><a href="{{ url_for('.send_email', service_id=service_id) }}">Send emails</a></li>-->
-    <!--<li><a href="{{ url_for('.view_jobs', service_id=service_id) }}">Activity</a></li>-->
+    <li><a href="{{ url_for('.send_sms', service_id=service_id) }}">Send text messages</a></li>
+    <li><a href="{{ url_for('.send_email', service_id=service_id) }}">Send emails</a></li>
+    <li><a href="{{ url_for('.view_jobs', service_id=service_id) }}">Activity</a></li>
     <li><a href="{{ url_for('.manage_service_templates', service_id=service_id) }}">Templates</a></li>
   </ul>
   <ul>
@@ -13,7 +13,7 @@
     <li><a href="{{ url_for('.api_keys', service_id=service_id) }}">API keys</a></li>
   </ul>
   <ul>
-    <!--<li><a href="{{ url_for('.manage_users', service_id=service_id) }}">Manage users</a></li>-->
+    <li><a href="{{ url_for('.manage_users', service_id=service_id) }}">Manage users</a></li>
     <li><a href="{{ url_for('.service_settings', service_id=service_id) }}">Service settings</a></li>
   </ul>
 </nav>

From 4e158e203cf9081d0f35d95a9c87b36fc9741982 Mon Sep 17 00:00:00 2001
From: Chris Hill-Scott <me@quis.cc>
Date: Tue, 26 Jan 2016 16:20:57 +0000
Subject: [PATCH 2/3] Update styleguide

---
 app/assets/stylesheets/components/table.scss |  3 ++-
 app/templates/admin_template.html            | 16 +++-------------
 app/templates/components/banner.html         |  8 +++++++-
 app/templates/views/styleguide.html          | 15 +++++++++++++--
 4 files changed, 25 insertions(+), 17 deletions(-)

diff --git a/app/assets/stylesheets/components/table.scss b/app/assets/stylesheets/components/table.scss
index ce6930370..f5c0c94b0 100644
--- a/app/assets/stylesheets/components/table.scss
+++ b/app/assets/stylesheets/components/table.scss
@@ -56,6 +56,7 @@
 .table-empty-message {
   @include core-16;
   color: $secondary-text-colour;
+  border-top: 1px solid $border-colour;
   border-bottom: 1px solid $border-colour;
-  padding: 5px 0 8px 0;
+  padding: 0.75em 0 0.5625em 0;
 }
diff --git a/app/templates/admin_template.html b/app/templates/admin_template.html
index f5907f644..42505fc90 100644
--- a/app/templates/admin_template.html
+++ b/app/templates/admin_template.html
@@ -72,19 +72,9 @@
   <main id="content" role="main" class="page-container">
       {% with messages = get_flashed_messages(with_categories=true) %}
         {% if messages %}
-          <ul class="banner-dangerous">
-            {% for category, message in messages %}
-              <li class="flash-message">
-                {{ message }}
-                {% if 'delete' == category %}
-                  <form method='post'>
-                    <input type="hidden" name="csrf_token" value="{{ csrf_token() }}" />
-                    <input type="submit" class="button" name="delete" value="Yes, delete this template" />
-                  </form>
-                {% endif %}
-              </li>
-            {% endfor %}
-          </ul>
+          {% for category, message in messages %}
+            {{ banner(message, 'dangerous', delete_button="Yes, delete this template" if 'delete' == category else None)}}
+          {% endfor %}
         {% endif %}
       {% endwith %}
       {% block fullwidth_content %}{% endblock %}
diff --git a/app/templates/components/banner.html b/app/templates/components/banner.html
index 39e0a1e5f..8adbfa5c6 100644
--- a/app/templates/components/banner.html
+++ b/app/templates/components/banner.html
@@ -1,5 +1,11 @@
-{% macro banner(body, type=None, with_tick=False) %}
+{% macro banner(body, type=None, with_tick=False, delete_button=None) %}
   <div class='banner{% if type %}-{{ type }}{% endif %}{% if with_tick %}-with-tick{% endif %}'>
     {{ body }}
+    {% if delete_button %}
+      <form method='post'>
+        <input type="hidden" name="csrf_token" value="{{ csrf_token() }}" />
+        <input type="submit" class="button" name="delete" value="{{ delete_button }}" />
+      </form>
+    {% endif %}
   </div>
 {% endmacro %}
diff --git a/app/templates/views/styleguide.html b/app/templates/views/styleguide.html
index edf2a753c..2dde4716a 100644
--- a/app/templates/views/styleguide.html
+++ b/app/templates/views/styleguide.html
@@ -22,14 +22,25 @@
   <p><a href="https://github.com/alphagov/notifications-admin/blob/master/app/templates/views/styleguide.html">View source</a></p>
 
   <h2 class="heading-large">Banner</h2>
-  <p>Used to show the result of a user’s action.</p>
-  {{ banner("This is a banner", with_tick=True) }}
+  <p>Used to show the status of a thing or action.</p>
+
+  {{ banner("You sent 1,234 text messages", with_tick=True) }}
+
   <div class="grid-row">
     <div class="column-one-third">
       {{ banner("Delivered 10:20") }}
     </div>
   </div>
 
+  {{ banner(
+    'Your service is in restricted mode. You can only send notifications to yourself.',
+    'info'
+  ) }}
+
+  {{ banner('You’re not allowed to do this', 'dangerous')}}
+
+  {{ banner('Are you sure you want to delete?', 'dangerous', delete_button="Yes, delete this thing")}}
+
   <h2 class="heading-large">Big number</h2>
 
   <p>Used to show some important statistics.</p>

From e03813ceefa73e3cb7871be75408238b36070b89 Mon Sep 17 00:00:00 2001
From: Chris Hill-Scott <me@quis.cc>
Date: Tue, 26 Jan 2016 16:26:08 +0000
Subject: [PATCH 3/3] Make 'API key was revoked' banner green

---
 app/assets/stylesheets/components/banner.scss | 3 ++-
 app/main/views/api_keys.py                    | 2 +-
 app/templates/admin_template.html             | 6 +++++-
 3 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/app/assets/stylesheets/components/banner.scss b/app/assets/stylesheets/components/banner.scss
index f6c08fba9..e690010f0 100644
--- a/app/assets/stylesheets/components/banner.scss
+++ b/app/assets/stylesheets/components/banner.scss
@@ -1,5 +1,6 @@
 %banner,
-.banner {
+.banner,
+.banner-default {
 
   @include core-19;
   background: $turquoise;
diff --git a/app/main/views/api_keys.py b/app/main/views/api_keys.py
index e2d77bf31..fd29426ae 100644
--- a/app/main/views/api_keys.py
+++ b/app/main/views/api_keys.py
@@ -51,5 +51,5 @@ def revoke_api_key(service_id, key_id):
         )
     elif request.method == 'POST':
         api_key_api_client.revoke_api_key(service_id=service_id, key_id=key_id)
-        flash('‘{}’ was revoked'.format(key_name))
+        flash('‘{}’ was revoked'.format(key_name), 'default')
         return redirect(url_for('.api_keys', service_id=service_id))
diff --git a/app/templates/admin_template.html b/app/templates/admin_template.html
index 42505fc90..346959cba 100644
--- a/app/templates/admin_template.html
+++ b/app/templates/admin_template.html
@@ -73,7 +73,11 @@
       {% with messages = get_flashed_messages(with_categories=true) %}
         {% if messages %}
           {% for category, message in messages %}
-            {{ banner(message, 'dangerous', delete_button="Yes, delete this template" if 'delete' == category else None)}}
+            {{ banner(
+              message,
+              'default' if category == 'default' else 'dangerous',
+              delete_button="Yes, delete this template" if 'delete' == category else None
+            )}}
           {% endfor %}
         {% endif %}
       {% endwith %}