this part is good (#1751)

TASVideos · Jan 24, 2024 · 828407a · 828407a
1 parent 1c8034d
commit 828407a
Show file tree

Hide file tree

Showing 5 changed files with 42 additions and 23 deletions.
diff --git a/TASVideos/Pages/Games/Goals/List.cshtml b/TASVideos/Pages/Games/Goals/List.cshtml
@@ -6,7 +6,7 @@
 }
 
 <h4>Game: <a asp-page="/Games/Index" asp-route-id="@Model.GameId">@Model.GameDisplayName</a>
-	<a condition="!string.IsNullOrWhiteSpace(returnUrl)" asp-page="/Goals/Index" class="btn btn-secondary mb-2 float-end"><span class="fa fa-arrow-left"></span> Back</a>
+	<a condition="!string.IsNullOrWhiteSpace(returnUrl)" href="@returnUrl" class="btn btn-secondary mb-2 float-end"><span class="fa fa-arrow-left"></span> Back</a>
 </h4>
 <table class="table table-bordered table-striped">
 	<tr>
@@ -33,24 +33,32 @@
 			<td condition="@(goal.Id != Model.GoalToEdit || goal.DisplayName == "baseline")">
 				@goal.DisplayName</td>
 			<td>
-				@foreach (var pub in goal.Publications.OrderBy(p => p.Obs).ThenByDescending(p => p.Id))
-				{
-					<small condition="pub.Obs">(Obsolete)</small>
-					<pub-link id="@pub.Id">
-						@pub.Title
-					</pub-link><br />
-				}
+				<ul>
+					@foreach (var pub in goal.Publications.OrderBy(p => p.Obs).ThenByDescending(p => p.Id))
+					{
+						<li>
+							<small condition="pub.Obs">(Obsolete)</small>
+							<pub-link id="@pub.Id">
+								[@pub.Id] @pub.Title
+							</pub-link>
+						</li>
+					}
+				</ul>
 			</td>
 			<td>
-				@foreach (var sub in goal.Submissions.OrderByDescending(s => s.Id))
-				{
-					<sub-link id="@sub.Id">@sub.Title</sub-link><br />
-				}
+				<ul>
+					@foreach (var sub in goal.Submissions.OrderByDescending(s => s.Id))
+					{
+						<li>
+							<sub-link id="@sub.Id">@sub.Title</sub-link>
+						</li>
+					}
+				</ul>
 			</td>
 			<td>
 				<div class="btn-toolbar">
-					<a asp-page="List" asp-route-goalToEdit="@goal.Id" permission="CatalogMovies" condition="@(goal.Id != Model.GoalToEdit && goal.DisplayName != "baseline")" asp-route-gameId="@Model.GameId" class="btn btn-primary"><i class="fa fa-pencil"></i> Edit</a>
-					<a condition="canDelete" asp-page-handler="Delete" asp-route-gameGroupId="@goal.Id" class="btn btn-danger"><i class="fa fa-remove"></i></a>
+					<a asp-page="List" asp-route-goalToEdit="@goal.Id" permission="CatalogMovies" condition="@(goal.Id != Model.GoalToEdit && goal.DisplayName != "baseline")" asp-route-gameId="@Model.GameId" class="btn btn-sm btn-primary"><i class="fa fa-pencil"></i> Edit</a>
+					<a condition="canDelete" asp-page-handler="Delete" asp-route-gameGroupId="@goal.Id" class="btn btn-sm btn-danger"><i class="fa fa-remove"></i> Delete</a>
 				</div>
 			</td>
 		</tr>

diff --git a/TASVideos/Pages/Games/Index.cshtml b/TASVideos/Pages/Games/Index.cshtml
@@ -125,9 +125,6 @@
 					</a>)
 				</span>
 			</li>
-			<li condition="Model.Game.PublicationCount > 0">
-				<a asp-page="PublicationHistory" asp-route-id="@Model.Game.Id">Publication History</a>
-			</li>
 			<li>
 				<a href="/Subs-List?GameId=@(Model.Game.Id)">
 					@Model.Game.SubmissionCount Submissions
@@ -137,7 +134,15 @@
 				<a href="/UserFiles/Game/@(Model.Game.Id)">
 					@Model.Game.UserFilesCount User Files
 				</a>
-			</li>
+            </li>
+            <li condition="Model.Game.PublicationCount > 0">
+                <a asp-page="PublicationHistory" asp-route-id="@Model.Game.Id">Publication History</a>
+            </li>
+            <li>
+				<a href="/Games/@(Model.Game.Id)/Goals/List">
+                    Goals
+                </a>
+            </li>
 		</ul>
 	</column>
 	<column md="6" condition="Model.WatchFiles.Any()">

diff --git a/TASVideos/Pages/Games/Index.cshtml.cs b/TASVideos/Pages/Games/Index.cshtml.cs
@@ -52,7 +52,7 @@ public async Task<IActionResult> OnGet()
 		Game = game;
 		var movies = await _db.Publications
 			.Where(p => p.GameId == Game.Id && p.ObsoletedById == null)
-			.OrderBy(p => p.GameGoal!.DisplayName.Length)
+			.OrderBy(p => p.GameGoal!.DisplayName == "baseline" ? -1 : p.GameGoal!.DisplayName.Length)
 			.ThenBy(p => p.Frames)
 			.Select(p => new
 			{
@@ -75,8 +75,14 @@ public async Task<IActionResult> OnGet()
 
 		Movies = movies
 			.Select(m => new TabMiniMovieModel(
-				movies.Count(mm => mm.Goal == m.Goal) > 1 ? m.GameTitle : "",
-				m.Goal,
+				movies.Count(mm => mm.Goal == m.Goal) > 1
+					? m.GameTitle
+					: m.Goal == "baseline"
+					? "(baseline)"
+					: "",
+				m.Goal == "baseline"
+					? ""
+					: m.Goal,
 				new MiniMovieModel
 				{
 					Id = m.Id,

diff --git a/TASVideos/Pages/Publications/Catalog.cshtml b/TASVideos/Pages/Publications/Catalog.cshtml
@@ -57,7 +57,7 @@
 						<span asp-validation-for="Catalog.GameGoalId" class="text-danger"></span>
 					</div>
 					<div class="col-2">
-						<button id="create-goal" type="button" class="btn btn-primary">New</button>
+						<button id="create-goal" type="button" class="btn btn-primary">Manage</button>
 					</div>
 				</row>
 			</fieldset>

diff --git a/TASVideos/Pages/Submissions/Catalog.cshtml b/TASVideos/Pages/Submissions/Catalog.cshtml
@@ -58,7 +58,7 @@
 						<span asp-validation-for="Catalog.GameGoalId" class="text-danger"></span>
 					</div>
 					<div class="col-2">
-						<button id="create-goal" type="button" class="btn btn-primary">New</button>
+						<button id="create-goal" type="button" class="btn btn-primary">Manage</button>
 					</div>
 				</row>
 			</fieldset>