From f5aee035b32c32e704c1c8c439fda89293db3eba Mon Sep 17 00:00:00 2001
From: rubenwardy <rw@rubenwardy.com>
Date: Tue, 4 May 2021 21:52:43 +0100
Subject: [PATCH] Use vertical nav

---
 app/templates/packages/create_edit.html      | 11 ++++
 app/templates/packages/edit_maintainers.html |  2 +-
 app/templates/packages/package_base.html     | 54 ++++++++++----------
 app/templates/packages/remove.html           |  2 +-
 app/templates/packages/view.html             |  2 +-
 5 files changed, 40 insertions(+), 31 deletions(-)

diff --git a/app/templates/packages/create_edit.html b/app/templates/packages/create_edit.html
index c9797cfb..c67af90b 100644
--- a/app/templates/packages/create_edit.html
+++ b/app/templates/packages/create_edit.html
@@ -25,6 +25,17 @@
 {% endblock %}
 
 {% block content %}
+	{% if package %}
+		<h2 class="mt-0">{{ _("Edit Details") }}</h2>
+	{% else %}
+		<h2 class="mt-0">
+			{{ _("Create Package") }}
+			{% if author != current_user %}
+				for {{ author.display_name }}
+			{% endif %}
+		</h2>
+	{% endif %}
+
 	{% if not package %}
 		<div class="alert alert-info">
 			<a class="float-right btn btn-sm btn-default" href="{{ url_for('flatpage', path='policy_and_guidance') }}">{{ _("View") }}</a>
diff --git a/app/templates/packages/edit_maintainers.html b/app/templates/packages/edit_maintainers.html
index c865910f..81d387fa 100644
--- a/app/templates/packages/edit_maintainers.html
+++ b/app/templates/packages/edit_maintainers.html
@@ -4,8 +4,8 @@
 	{{ _("Edit Maintainers") }}
 {% endblock %}
 
-
 {% block content %}
+	<h2 class="mt-0">{{ _("Maintainers") }}</h2>
 	{% from "macros/forms.html" import render_submit_field, render_field %}
 	<p>
 		{{ _("Maintainers are given write access to the package.") }}
diff --git a/app/templates/packages/package_base.html b/app/templates/packages/package_base.html
index 18b39fc6..4794558f 100644
--- a/app/templates/packages/package_base.html
+++ b/app/templates/packages/package_base.html
@@ -1,40 +1,38 @@
 {% extends "base.html" %}
 
 {% block container %}
+<main class="container mt-4">
 	{% if tabs %}
-		<nav class="pt-4 tabs-container">
-			<div class="container">
-				<a class="float-right btn btn-primary" href="{{ package.getDetailsURL() }}">
-					{{ _("View Package") }}
-				</a>
-				<h1 class="mb-5">
-					<a href="{{ package.getDetailsURL() }}">
-						{{ package.title }}
+		<div class="row">
+			<div class="col-md-3 mb-4">
+				<div class="list-group">
+					<a class="list-group-item list-group-item-action" href="{{ package.getDetailsURL() }}">
+						<span class="row m-0 p-0">
+							<span class="col-auto m-0 p-0">
+								<img class="img-fluid user-photo img-thumbnail img-thumbnail-1"
+										src="{{ package.getThumbnailOrPlaceholder(1) }}" alt="Thumbnail" style="max-height: 20px;">
+							</span>
+							<span class="col m-0 p-0 pl-2">
+								{{ package.title }}
+							</span>
+						</span>
 					</a>
-				</h1>
-
-				<ul class="nav nav-tabs">
 					{% for item in tabs %}
-						<li class="nav-item">
-							<a href="{{ item.url }}" class="nav-link {% if item.id == current_tab %}active{% endif %}">
-								{{ item.title }}
-							</a>
-						</li>
+						<a href="{{ item.url }}"
+								class="list-group-item list-group-item-action {% if item.id == current_tab %}active{% endif %}">
+							{{ item.title }}
+						</a>
 					{% endfor %}
-				</ul>
+				</div>
 			</div>
-		</nav>
-	{% endif %}
+			<div class="col-md-9">
+		{% endif %}
 
-	{% if tabs %}
-		<main class="container mt-5">
-			{{ self.content() }}
-		</main>
-	{% else %}
-		<main class="container mt-4">
-			<h1 class="mb-4">{{ self.title() }}</h1>
+		{{ self.content() }}
 
-			{{ self.content() }}
-		</main>
+		{% if tabs %}
+			</div>
+		</div>
 	{% endif %}
+</main>
 {% endblock %}
diff --git a/app/templates/packages/remove.html b/app/templates/packages/remove.html
index f8394f81..0d5b43b4 100644
--- a/app/templates/packages/remove.html
+++ b/app/templates/packages/remove.html
@@ -6,7 +6,7 @@ Remove {{ package.title }}
 
 {% block content %}
 	<form method="POST" action="">
-		<h2>Remove {{ package.title }}</h2>
+		<h2 class="mt-0">Remove {{ package.title }}</h2>
 		<p>
 			In order to avoid data loss, you cannot permanently delete packages.
 			You can remove them from ContentDB, which will cause them to not be
diff --git a/app/templates/packages/view.html b/app/templates/packages/view.html
index dfeebcd0..fc09dfec 100644
--- a/app/templates/packages/view.html
+++ b/app/templates/packages/view.html
@@ -454,6 +454,7 @@
 					{% from "macros/releases.html" import render_releases %}
 					{{ render_releases(releases, package, current_user) }}
 				</ul>
+
 				<h3>
 					{% if package.approved and package.checkPerm(current_user, "CREATE_THREAD") %}
 						<div class="btn-group float-right">
@@ -462,7 +463,6 @@
 					{% endif %}
 					Threads
 				</h3>
-
 				<div class="list-group">
 					{% from "macros/threads.html" import render_compact_threadlist %}
 					{{ render_compact_threadlist(threads) }}