From 5dcff01436bf81cfbff8085fbae4692ec57a9fdf Mon Sep 17 00:00:00 2001
From: rubenwardy <rw@rubenwardy.com>
Date: Sat, 22 Dec 2018 12:20:26 +0000
Subject: [PATCH] Improve button colours and position in package view

---
 app/scss/custom.scss             | 11 +++++++++++
 app/templates/packages/view.html | 20 +++++++++-----------
 2 files changed, 20 insertions(+), 11 deletions(-)

diff --git a/app/scss/custom.scss b/app/scss/custom.scss
index c95ebfd..6663485 100644
--- a/app/scss/custom.scss
+++ b/app/scss/custom.scss
@@ -20,3 +20,14 @@
 .alert .btn {
     text-decoration: none;
 }
+
+.btn-download {
+    color: #fff;
+    background-color: #00b05c;
+    border-color: #00b05c;
+}
+
+.btn-download:focus, .btn-download.focus {
+    -webkit-box-shadow: 0 0 0 0.2rem rgba(231, 76, 60, 0.5);
+    box-shadow: 0 0 0 0.2rem rgba(231, 76, 60, 0.5);
+}
diff --git a/app/templates/packages/view.html b/app/templates/packages/view.html
index 358fdfc..30ca424 100644
--- a/app/templates/packages/view.html
+++ b/app/templates/packages/view.html
@@ -122,7 +122,7 @@
 
 		<aside class="float-right ml-4" style="width: 18rem;">
 			{% if package.getDownloadRelease() %}
-				<a class="btn btn-success btn-lg btn-block"
+				<a class="btn btn-download btn-lg btn-block"
 						href="{{ package.getDownloadURL() }}" class="btn_green">
 					Download
 				</a>
@@ -135,13 +135,13 @@
 					Details
 					<div class="btn-group float-right">
 					{% if package.checkPerm(current_user, "EDIT_PACKAGE") %}
-						<a class="btn btn-primary btn-sm btn-xs mx-1" href="{{ package.getEditURL() }}">Edit</a>
+						<a class="btn btn-default btn-sm mx-1" href="{{ package.getEditURL() }}">Edit</a>
 					{% endif %}
 					{# {% if current_user.is_authenticated %}
-						<a class="btn btn-primary btn-sm btn-xs mx-1" href="{{ package.getCreateEditRequestURL() }}">Suggest Changes</a>
+						<a class="btn btn-default btn-sm mx-1" href="{{ package.getCreateEditRequestURL() }}">Suggest Changes</a>
 					{% endif %} #}
 					{% if package.checkPerm(current_user, "DELETE_PACKAGE") %}
-						<a class="btn btn-danger btn-sm btn-xs mx-1" href="{{ package.getDeleteURL() }}">Delete</a>
+						<a class="btn btn-danger btn-sm mx-1" href="{{ package.getDeleteURL() }}">Delete</a>
 					{% endif %}
 					</div>
 
@@ -251,8 +251,8 @@
 				<div class="card-header">
 					Releases
 					{% if package.checkPerm(current_user, "MAKE_RELEASE") %}
-						<a class="btn btn-xs btn-primary float-right"
-						href="{{ package.getCreateReleaseURL() }}">+</a>
+						<a class="float-right"
+							href="{{ package.getCreateReleaseURL() }}">+</a>
 					{% endif %}
 				</div>
 				<ul class="list-group list-group-flush">
@@ -291,13 +291,11 @@
 
 			<div class="card my-4"">
 				<div class="card-header">
-					Threads
 				{% if package.approved and package.checkPerm(current_user, "CREATE_THREAD") %}
-					<a class="btn btn-xs btn-primary float-right"
-							href="{{ url_for('new_thread_page', pid=package.id) }}">
-						+
-					</a>
+					<a class="float-right"
+						href="{{ url_for('new_thread_page', pid=package.id) }}">+</a>
 				{% endif %}
+					Threads
 				</div>
 				<ul class="list-group list-group-flush">
 					{% from "macros/threads.html" import render_threadlist %}