Skip to content

Commit e2ac04a

Browse files
committed
Merge av-02: Complete setup progress persistence implementation
2 parents 8a96ed5 + 6b93691 commit e2ac04a

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

54 files changed

+2863
-1300
lines changed

frontend/static/css/responsive-fix.css

Lines changed: 2 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -17,10 +17,7 @@
1717
border: none !important;
1818
}
1919

20-
/* Better handling of the Save button in the settings header */
21-
.settings-actions {
22-
margin-left: auto;
23-
}
20+
/* Auto-save enabled - save button removed */
2421

2522
/* Mobile-specific settings adjustments */
2623
@media (max-width: 768px) {
@@ -55,20 +52,7 @@
5552
flex-wrap: wrap !important;
5653
}
5754

58-
/* Move Save button to its own row when space is limited - moved from tablet view */
59-
#settingsSection .settings-actions {
60-
margin-top: 10px;
61-
width: 100%;
62-
display: flex;
63-
justify-content: flex-end;
64-
}
65-
66-
/* Fix the positioning of the Save button on narrow screens - moved from tablet view */
67-
#saveSettingsButton {
68-
position: relative !important;
69-
top: auto !important;
70-
right: auto !important;
71-
}
55+
/* Auto-save enabled - save button styles removed */
7256

7357
/* Fix General Settings section header on mobile */
7458
#settingsSection .section-header {

frontend/static/js/apps.js

Lines changed: 145 additions & 241 deletions
Large diffs are not rendered by default.

frontend/static/js/apps/whisparr.js

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -195,12 +195,3 @@ function setupWhisparrForm() {
195195
}
196196
}
197197

198-
// Helper function for escaping HTML (keep if needed elsewhere, e.g., if logs are added here later)
199-
function escapeHtml(unsafe) {
200-
return unsafe
201-
.replace(/&/g, "&")
202-
.replace(/</g, "&lt;")
203-
.replace(/>/g, "&gt;")
204-
.replace(/"/g, "&quot;")
205-
.replace(/'/g, "&#039;");
206-
}

frontend/static/js/direct-reset.js

Lines changed: 0 additions & 191 deletions
This file was deleted.

frontend/static/js/logs.js

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -279,11 +279,7 @@ window.LogsModule = {
279279
// Set up polling with user's configured interval
280280
this.setupLogPolling(appType);
281281

282-
// Update connection status
283-
if (this.elements.logConnectionStatus) {
284-
this.elements.logConnectionStatus.textContent = 'Connected';
285-
this.elements.logConnectionStatus.className = 'status-connected';
286-
}
282+
// Status will be updated by loadLogsFromAPI on success/failure
287283
},
288284

289285
// Set up log polling with user's configured interval
@@ -335,6 +331,11 @@ window.LogsModule = {
335331
.then(data => {
336332
if (data.success && data.logs) {
337333
this.processLogsFromAPI(data.logs, appType, isPolling);
334+
// Update connection status on successful API call (only on initial load, not polling)
335+
if (this.elements.logConnectionStatus && !isPolling) {
336+
this.elements.logConnectionStatus.textContent = 'Connected';
337+
this.elements.logConnectionStatus.className = 'status-connected';
338+
}
338339
} else {
339340
console.error('[LogsModule] Failed to load logs:', data.error || 'No logs in response');
340341
if (this.elements.logConnectionStatus) {

0 commit comments

Comments
 (0)