From: Michael Tremer <michael.tremer@ipfire.org>
Date: Tue, 23 May 2023 16:19:40 +0000 (+0000)
Subject: builders: Rename list page to index for consistency
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=7e5a1249c7e164faf57012da2870c9f8d9f9b8a5;p=pbs.git

builders: Rename list page to index for consistency

Signed-off-by: Michael Tremer <michael.tremer@ipfire.org>
---

diff --git a/Makefile.am b/Makefile.am
index 9974a933..5f0c1133 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -169,7 +169,7 @@ dist_templates_builders_DATA = \
 	src/templates/builders/delete.html \
 	src/templates/builders/detail.html \
 	src/templates/builders/edit.html \
-	src/templates/builders/list.html \
+	src/templates/builders/index.html \
 	src/templates/builders/new.html
 
 templates_buildersdir = $(templatesdir)/builders
diff --git a/src/templates/builders/list.html b/src/templates/builders/index.html
similarity index 78%
rename from src/templates/builders/list.html
rename to src/templates/builders/index.html
index 74c78b6e..fa101b6c 100644
--- a/src/templates/builders/list.html
+++ b/src/templates/builders/index.html
@@ -3,26 +3,24 @@
 {% block title %}{{ _("Builders") }}{% end block %}
 
 {% block body %}
-	<section class="section">
-		<div class="container">
-			<nav class="breadcrumb" aria-label="breadcrumbs">
-				<ul>
-					<li class="is-active">
-						<a href="#" aria-current="page">{{ _("Builders") }}</a>
-					</li>
-				</ul>
-			</nav>
-
-			<h1 class="title is-1">{{ _("Builders") }}</h1>
+	<section class="hero is-light">
+		<div class="hero-body">
+			<div class="container">
+				<nav class="breadcrumb" aria-label="breadcrumbs">
+					<ul>
+						<li class="is-active">
+							<a href="#" aria-current="page">{{ _("Builders") }}</a>
+						</li>
+					</ul>
+				</nav>
 
-			{% if current_user and current_user.is_admin() %}
-				<div class="block">
-					<a class="button is-success" href="/builders/new">
-						{{ _("Create Builder") }}
-					</a>
-				</div>
-			{% end %}
+				<h1 class="title is-1">{{ _("Builders") }}</h1>
+			</div>
+		</div>
+	</section>
 
+	<section class="section">
+		<div class="container">
 			{% for builder in builders %}
 				<div class="block">
 					<div class="box">
@@ -45,6 +43,12 @@
 					</div>
 				</div>
 			{% end %}
+
+			{% if current_user and current_user.is_admin() %}
+				<a class="button is-success" href="/builders/new">
+					{{ _("Create Builder") }}
+				</a>
+			{% end %}
 		</div>
 	</section>
 
diff --git a/src/web/__init__.py b/src/web/__init__.py
index e82dfb82..221769b7 100644
--- a/src/web/__init__.py
+++ b/src/web/__init__.py
@@ -177,7 +177,7 @@ class Application(tornado.web.Application):
 				jobs.APIv1LogStreamHandler),
 
 			# Builders
-			(r"/builders", builders.BuilderListHandler),
+			(r"/builders", builders.IndexHandler),
 			(r"/builders/new", builders.BuilderNewHandler),
 			(r"/builders/([A-Za-z0-9\-\.]+)/delete", builders.BuilderDeleteHandler),
 			(r"/builders/([A-Za-z0-9\-\.]+)/edit", builders.BuilderEditHandler),
diff --git a/src/web/builders.py b/src/web/builders.py
index 11f6b55e..3cd940e7 100644
--- a/src/web/builders.py
+++ b/src/web/builders.py
@@ -74,10 +74,9 @@ class StatsHandler(base.BackendMixin, tornado.websocket.WebSocketHandler):
 		})
 
 
-
-class BuilderListHandler(base.BaseHandler):
+class IndexHandler(base.BaseHandler):
 	def get(self):
-		self.render("builders/list.html", builders=self.backend.builders)
+		self.render("builders/index.html", builders=self.backend.builders)
 
 
 class BuilderDetailHandler(base.BaseHandler):