From de67c5316d33aec5a0df09ea0f7cc78e82d181c1 Mon Sep 17 00:00:00 2001
From: emeric <itmfr@yahoo.fr>
Date: Thu, 3 Sep 2020 10:54:15 +0200
Subject: [PATCH] Remove header title from main views, and set them active in
 the navbar

---
 approot/artists.xml           |  1 -
 approot/playqueue.xml         |  1 -
 approot/releases.xml          |  1 -
 approot/templates.xml         |  8 ++++----
 approot/tracks.xml            |  1 -
 src/lms/ui/LmsApplication.cpp | 16 +++++++++++-----
 6 files changed, 15 insertions(+), 13 deletions(-)

diff --git a/approot/artists.xml b/approot/artists.xml
index 70516b79..2f29fbb6 100644
--- a/approot/artists.xml
+++ b/approot/artists.xml
@@ -4,7 +4,6 @@
 
 <message id="Lms.Explore.Artists.template">
 	<div class="Lms-header">
-		<h2>${tr:Lms.Explore.artists}</h2>
 		${mode class="nav nav-pills Lms-explore-mode-container"}
 		<div class="row">
 			<div class="col-sm-2">
diff --git a/approot/playqueue.xml b/approot/playqueue.xml
index 2eeb3268..c121fd6a 100644
--- a/approot/playqueue.xml
+++ b/approot/playqueue.xml
@@ -11,7 +11,6 @@
 	<div class="row">
 		<div class="col-lg-8">
 			<div class="Lms-header">
-				<h2>${tr:Lms.PlayQueue.playqueue}</h2>
 				${clear-btn}${shuffle-btn}${repeat-btn}${radio-btn}
 				<h4><small>${nb-tracks}</small></h4>
 			</div>
diff --git a/approot/releases.xml b/approot/releases.xml
index 3d9ec83c..091dcd34 100644
--- a/approot/releases.xml
+++ b/approot/releases.xml
@@ -4,7 +4,6 @@
 
 <message id="Lms.Explore.Releases.template">
 	<div class="Lms-header">
-		<h2>${tr:Lms.Explore.releases}</h2>
 		${mode class="nav nav-pills Lms-explore-mode-container"}
 		${play-btn}${more-btn}
 	</div>
diff --git a/approot/templates.xml b/approot/templates.xml
index 5be1e07d..942a0770 100644
--- a/approot/templates.xml
+++ b/approot/templates.xml
@@ -18,10 +18,10 @@
 			</div>
 			<div class="collapse navbar-collapse">
 				<ul class="nav navbar-nav">
-					<li>${artists class="Lms-btn"}</li>
-					<li>${releases class="Lms-btn"}</li>
-					<li>${tracks class="Lms-btn"}</li>
-					<li>${playqueue class="Lms-btn"}</li>
+					<li class="${artists-active}">${artists class="Lms-btn"}</li>
+					<li class="${releases-active}">${releases class="Lms-btn"}</li>
+					<li class="${tracks-active}">${tracks class="Lms-btn"}</li>
+					<li class="${playqueue-active}">${playqueue class="Lms-btn"}</li>
 				</ul>
 				<div class="navbar-form navbar-left">
 					${filters class="form-group Lms-clusters-control"}
diff --git a/approot/tracks.xml b/approot/tracks.xml
index 81a90d2c..846b8184 100644
--- a/approot/tracks.xml
+++ b/approot/tracks.xml
@@ -7,7 +7,6 @@
 	<div class="row">
 		<div class="col-lg-8">
 			<div class="Lms-header">
-				<h2>${tr:Lms.Explore.tracks}</h2>
 				${mode class="nav nav-pills Lms-explore-mode-container"}
 				${play-btn}${more-btn}
 			</div>
diff --git a/src/lms/ui/LmsApplication.cpp b/src/lms/ui/LmsApplication.cpp
index 67481965..7327b061 100644
--- a/src/lms/ui/LmsApplication.cpp
+++ b/src/lms/ui/LmsApplication.cpp
@@ -347,8 +347,9 @@ enum IdxRoot
 	IdxAdminUser,
 };
 
-static void
-handlePathChange(Wt::WStackedWidget* stack, bool isAdmin)
+static
+void
+handlePathChange(Wt::WTemplate& main, Wt::WStackedWidget& stack, bool isAdmin)
 {
 	static const struct
 	{
@@ -372,6 +373,11 @@ handlePathChange(Wt::WStackedWidget* stack, bool isAdmin)
 
 	LMS_LOG(UI, DEBUG) << "Internal path changed to '" << wApp->internalPath() << "'";
 
+	main.bindString("artists-active", wApp->internalPathMatches("/artists") ? "active" : "");
+	main.bindString("releases-active", wApp->internalPathMatches("/releases") ? "active" : "");
+	main.bindString("tracks-active", wApp->internalPathMatches("/tracks") ? "active" : "");
+	main.bindString("playqueue-active", wApp->internalPathMatches("/playqueue") ? "active" : "");
+
 	for (const auto& view : views)
 	{
 		if (wApp->internalPathMatches(view.path))
@@ -379,7 +385,7 @@ handlePathChange(Wt::WStackedWidget* stack, bool isAdmin)
 			if (view.admin && !isAdmin)
 				break;
 
-			stack->setCurrentIndex(view.index);
+			stack.setCurrentIndex(view.index);
 			return;
 		}
 	}
@@ -597,10 +603,10 @@ LmsApplication::createHome()
 
 	internalPathChanged().connect([=]
 	{
-		handlePathChange(mainStack, isUserAdmin());
+		handlePathChange(*main, *mainStack, isUserAdmin());
 	});
 
-	handlePathChange(mainStack, isUserAdmin());
+	handlePathChange(*main, *mainStack, isUserAdmin());
 }
 
 void