From 3a0e0377f9bf461615358b3b72e5192ffa43334b Mon Sep 17 00:00:00 2001
From: rubenwardy <rw@rubenwardy.com>
Date: Fri, 21 Dec 2018 17:00:16 +0000
Subject: [PATCH] Improve button placement

---
 app/templates/packages/view.html | 26 ++++++++++++++++----------
 1 file changed, 16 insertions(+), 10 deletions(-)

diff --git a/app/templates/packages/view.html b/app/templates/packages/view.html
index e214042..6ca5a0c 100644
--- a/app/templates/packages/view.html
+++ b/app/templates/packages/view.html
@@ -21,11 +21,6 @@
 
 			<div class="row" style="margin-top: 4rem;">
 				<div class="col">
-					{% if package.getDownloadRelease() %}
-						<a class="btn btn-success" href="{{ package.getDownloadURL() }}" class="btn_green">Download</a>
-					{% else %}
-						No download available.
-					{% endif %}
 				</div>
 				<div class="btn-group-horizontal col-md-auto">
 					{% if package.repo %}<a class="btn btn-primary" href="{{ package.repo }}">View Source</a>{% endif %}
@@ -35,9 +30,6 @@
 					{# {% if current_user.is_authenticated %}
 						<a class="btn btn-primary" href="{{ package.getCreateEditRequestURL() }}">Suggest Changes</a>
 					{% endif %} #}
-					{% if package.checkPerm(current_user, "DELETE_PACKAGE") %}
-						<a class="btn btn-danger" href="{{ package.getDeleteURL() }}">Delete</a>
-					{% endif %}
 				</div>
 			</div>
 		</div>
@@ -132,12 +124,26 @@
 		{% endif %}
 
 		<aside class="float-right ml-4" style="width: 18rem;">
-			<div class="card">
+			{% if package.getDownloadRelease() %}
+				<a class="btn btn-success btn-lg btn-block"
+						href="{{ package.getDownloadURL() }}" class="btn_green">
+					Download
+				</a>
+			{% else %}
+				No download available.
+			{% endif %}
+
+			<div class="card my-4">
 				<div class="card-header">
 					Details
+					<div class="btn-group float-right">
 					{% if package.checkPerm(current_user, "EDIT_PACKAGE") %}
-						<a class="btn btn-sm btn-xs btn-primary float-right" href="{{ package.getEditURL() }}">Edit</a>
+						<a class="btn btn-primary btn-sm btn-xs mx-1" href="{{ package.getEditURL() }}">Edit</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>
+					{% endif %}
+					</div>
 
 				</div>
 				<div class="card-body">