From 765039e778735bde327c3035c12f2fecb8d7c624 Mon Sep 17 00:00:00 2001
From: unkn <1+unkn@noreply.localhost>
Date: Mon, 28 Sep 2026 13:03:12 +0100
Subject: [PATCH] Update dashboard.py

Tidied the dashboard by adding a user drop down menu rather than having all users displayed on a single page
---
 dashboard.py | 87 ++++++++++++++++++++++++++++++++++++++++++++--------
 1 file changed, 74 insertions(+), 13 deletions(-)

diff --git a/dashboard.py b/dashboard.py
index 4b46f5c..9f5ad09 100644
--- a/dashboard.py
+++ b/dashboard.py
@@ -1,11 +1,9 @@
-import os
 import sqlite3
 from datetime import datetime
 import traceback
 
-# Configurable paths via environment variables with fallback defaults
-DB_PATH = os.getenv('NAVIDROME_DB_PATH', './navidrome.db')
-OUTPUT_HTML = os.getenv('OUTPUT_HTML_PATH', './public/index.html')
+DB_PATH = '/nddata/navidrome.db'
+OUTPUT_HTML = '/app/public/index.html'
 
 def get_all_users():
     try:
@@ -56,9 +54,17 @@ def generate_html():
     <title>Music Listening Dashboard - All-Time Top Tracks</title>
     <style>
         body { font-family: -apple-system, BlinkMacSystemFont, "Segoe UI", Roboto, sans-serif; background: #121212; color: #e0e0e0; margin: 0; padding: 20px; }
-        h1 { text-align: center; color: #fff; margin-bottom: 30px; }
-        .container { display: grid; grid-template-columns: repeat(auto-fit, minmax(400px, 1fr)); gap: 30px; max-width: 1800px; margin: 0 auto; }
-        .card { background: #1e1e1e; border-radius: 10px; padding: 25px; box-shadow: 0 4px 10px rgba(0,0,0,0.4); }
+        .header-container { display: flex; justify-content: space-between; align-items: center; max-width: 1200px; margin: 0 auto 20px auto; flex-wrap: wrap; gap: 15px; }
+        h1 { color: #fff; margin: 0; font-size: 24px; }
+        .controls { display: flex; align-items: center; gap: 15px; }
+        select { background: #1e1e1e; color: #e0e0e0; border: 1px solid #333; padding: 8px 12px; font-size: 14px; border-radius: 6px; outline: none; cursor: pointer; }
+        select:focus { border-color: #bb86fc; }
+        .refresh-btn { background: #bb86fc; color: #121212; border: none; padding: 9px 16px; font-size: 14px; font-weight: bold; border-radius: 6px; cursor: pointer; transition: background 0.2s; }
+        .refresh-btn:hover { background: #9965f4; }
+        .refresh-btn:disabled { background: #555; color: #888; cursor: not-allowed; }
+        .container { max-width: 1200px; margin: 0 auto; }
+        .card { background: #1e1e1e; border-radius: 10px; padding: 25px; box-shadow: 0 4px 10px rgba(0,0,0,0.4); display: none; }
+        .card.active { display: block; }
         h2 { border-bottom: 2px solid #333; padding-bottom: 10px; color: #bb86fc; margin-top: 0; }
         table { width: 100%; border-collapse: collapse; margin-top: 15px; }
         th, td { text-align: left; padding: 10px; border-bottom: 1px solid #2c2c2c; font-size: 14px; }
@@ -68,21 +74,40 @@ def generate_html():
         .count { font-size: 13px; color: #aaa; margin-bottom: 15px; }
         @media (max-width: 600px) {
             body { padding: 10px; }
-            .container { grid-template-columns: 1fr; }
+            .header-container { flex-direction: column; align-items: stretch; text-align: center; }
+            .controls { justify-content: center; }
             .card { padding: 15px; }
         }
     </style>
 </head>
 <body>
-    <h1>All-Time Top Tracks & History</h1>
+    <div class="header-container">
+        <h1>All-Time Top Tracks & History</h1>
+        <div class="controls">
+            <div>
+                <label for="userSelect" style="margin-right: 8px; font-size: 14px; color: #888;">User:</label>
+                <select id="userSelect" onchange="switchUser()">
+"""
+
+    for i, user in enumerate(users):
+        selected = "selected" if i == 0 else ""
+        html_content += f'                    <option value="{user}" {selected}>{user}</option>\n'
+
+    html_content += """                </select>
+            </div>
+            <button id="refreshBtn" class="refresh-btn" onclick="triggerRefresh()">Regenerate Stats</button>
+        </div>
+    </div>
+
     <div class="container">
 """
 
-    for user in users:
+    for i, user in enumerate(users):
         print(f"Processing scrobbles for {user}...")
         rows = get_aggregated_scrobbles(user)
+        active_class = " active" if i == 0 else ""
         html_content += f"""
-        <div class="card">
+        <div class="card{active_class}" id="user-card-{user}">
             <h2>{user}</h2>
             <div class="count">Unique tracks played: {len(rows)}</div>
             <table>
@@ -102,15 +127,51 @@ def generate_html():
 
     html_content += """
     </div>
+
+    <script>
+        function switchUser() {
+            const selectedUser = document.getElementById('userSelect').value;
+            const cards = document.querySelectorAll('.card');
+            cards.forEach(card => {
+                card.classList.remove('active');
+            });
+            const targetCard = document.getElementById('user-card-' + selectedUser);
+            if (targetCard) {
+                targetCard.classList.add('active');
+            }
+        }
+
+        async function triggerRefresh() {
+            const btn = document.getElementById('refreshBtn');
+            btn.disabled = true;
+            btn.textContent = 'Regenerating...';
+            
+            try {
+                const response = await fetch('http://' + window.location.hostname + ':8083/', { method: 'POST' });
+                if (response.ok) {
+                    alert('Dashboard updated successfully! Reloading page...');
+                    location.reload();
+                } else {
+                    alert('Failed to regenerate stats.');
+                    btn.disabled = false;
+                    btn.textContent = 'Regenerate Stats';
+                }
+            } catch (err) {
+                console.error(err);
+                alert('Error connecting to the update server.');
+                btn.disabled = false;
+                btn.textContent = 'Regenerate Stats';
+            }
+        }
+    </script>
 </body>
 </html>
 """
 
     print(f"Writing HTML to {OUTPUT_HTML}...")
-    os.makedirs(os.path.dirname(OUTPUT_HTML), exist_ok=True)
     with open(OUTPUT_HTML, 'w') as f:
         f.write(html_content)
-    print("Dashboard HTML generated successfully for all users.")
+    print("Dashboard HTML generated successfully.")
 
 if __name__ == '__main__':
     try: