refactor: remove session filter dropdown and update session history table logic

Author: murdercode

src/index.html

@@ -363,12 +363,6 @@ <h4 id="selected-day-title">Today's Sessions</h4>
             <div class="sessions-header">
               <h3>Session History</h3>
               <div class="sessions-controls">
-                <select id="sessions-filter-period" class="filter-select">
-                  <option value="today" selected>Today</option>
-                  <option value="week">This Week</option>
-                  <option value="month">This Month</option>
-                  <option value="all">All Time</option>
-                </select>
                 <button id="export-sessions-btn" class="export-btn" title="Export to Excel">
                   <svg width="16" height="16" viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
                     <path

src/managers/navigation-manager.js

@@ -736,9 +736,9 @@ true // All sessions are focus sessions now
         } catch (error) {
             console.error('Failed to load session details:', error);
             const errorItem = document.createElement('div');
-            errorItem.className = 'sessions-empty';
+            errorItem.className = 'timeline-empty';
             errorItem.textContent = 'Error loading session data';
-            sessionsList.appendChild(errorItem);
+            timelineTrack.appendChild(errorItem);
         }
     }
 
@@ -846,6 +846,9 @@ true // All sessions are focus sessions now
         await this.updateWeeklySessionsChart();
         this.updateDailyChart();
         await this.updateTagUsageChart();
+        
+        // Update session history table for selected date
+        await this.populateSessionsTableForDate(date);
     }
 
     updateDailyDetails(date = this.currentDate) {
@@ -942,6 +945,9 @@ true // All sessions are focus sessions now
 
         // Check if this session is from today
         const isToday = this.isSameDay(date, new Date());
+        
+        // Define session type for display
+        const sessionType = session.session_type || session.type || 'Focus';
 
         // Parse start and end times
         const [startHour, startMinute] = session.start_time.split(':').map(Number);
@@ -1503,18 +1509,11 @@ true // All sessions are focus sessions now
 
     // Sessions History Table Methods
     async initSessionsTable() {
-        await this.populateSessionsTable('today');
+        await this.populateSessionsTableForDate(this.currentDate);
         this.setupSessionsTableEventListeners();
     }
 
     setupSessionsTableEventListeners() {
-        const filterSelect = document.getElementById('sessions-filter-period');
-        if (filterSelect) {
-            filterSelect.addEventListener('change', async (e) => {
-                await this.populateSessionsTable(e.target.value);
-            });
-        }
-
         const exportBtn = document.getElementById('export-sessions-btn');
         if (exportBtn) {
             exportBtn.addEventListener('click', () => {
@@ -1596,6 +1595,33 @@ true // All sessions are focus sessions now
         return allSessions;
     }
 
+    async populateSessionsTableForDate(date) {
+        const tableBody = document.getElementById('sessions-table-body');
+        if (!tableBody || !window.sessionManager) return;
+
+        const sessions = window.sessionManager.getSessionsForDate(date);
+        tableBody.innerHTML = '';
+
+        if (sessions.length === 0) {
+            tableBody.innerHTML = `
+                <tr>
+                    <td colspan="5" class="sessions-table-empty">
+                        No sessions found for selected date
+                    </td>
+                </tr>
+            `;
+            return;
+        }
+
+        // Sort sessions by time (newest first)
+        sessions.sort((a, b) => b.start_time.localeCompare(a.start_time));
+
+        for (const session of sessions) {
+            const row = await this.createSessionTableRow(session);
+            tableBody.appendChild(row);
+        }
+    }
+
     async createSessionTableRow(session) {
         const row = document.createElement('tr');
 
@@ -1700,9 +1726,8 @@ true // All sessions are focus sessions now
             await window.sessionManager.saveSessionsToStorage();
 
             // Refresh the table
-            const filterSelect = document.getElementById('sessions-filter-period');
-            const currentPeriod = filterSelect ? filterSelect.value : 'today';
-            await this.populateSessionsTable(currentPeriod);
+            const currentDate = this.selectedDate || this.currentDate;
+            await this.populateSessionsTableForDate(currentDate);
 
             // Refresh other views (with error handling for each)
             try {
@@ -1774,9 +1799,8 @@ true // All sessions are focus sessions now
 
     async exportSessionsToExcel() {
         try {
-            const filterSelect = document.getElementById('sessions-filter-period');
-            const currentPeriod = filterSelect ? filterSelect.value : 'today';
-            const sessions = this.getSessionsForPeriod(currentPeriod);
+            const currentDate = this.selectedDate || this.currentDate;
+            const sessions = window.sessionManager.getSessionsForDate(currentDate);
 
             if (sessions.length === 0) {
                 alert('No sessions to export for the selected period.');

src/utils/theme-loader.js

@@ -39,7 +39,7 @@ class ThemeLoader {
         // that gets updated by the build process or manually maintained
 
         // This could be enhanced to use a build-time script that generates this list
-                                                                                                                                                                                                                                                                        const knownThemes = [
+                                                                                                                                                                                                                                                                                const knownThemes = [
             'espresso.css',
             'pipboy.css',
             'pommodore64.css'