Skip to content

Enhance exercise record management #112

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
rodrigogiraoserrao merged 1 commit into from
Feb 11, 2026
Merged

Enhance exercise record management #112

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
32 changes: 27 additions & 5 deletions workout-exercise-record.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -142,6 +142,11 @@
font-weight: 600;
margin-bottom: 0.25rem;
}

.records-context {
margin: 0;
font-weight: 600;
}
</style>
</head>
<body>
Expand Down Expand Up @@ -173,13 +178,15 @@ <h2>Records</h2>
<button id="refresh-exercises">Refresh list</button>
</div>
</div>
<p id="records-context" class="status-text records-context">No exercise selected.</p>
<div class="record-table-container">
<div id="load-status" class="status"></div>
<table class="record-table">
<caption id="records-caption" aria-live="polite">Select an exercise to view records</caption>
<thead><tr><th>Weight</th><th>Reps</th></tr></thead>
<tbody id="record-rows"></tbody>
</table>
<p id="no-record-note" class="status-text" aria-live="polite"></p>
</div>
</section>

Expand Down Expand Up @@ -218,6 +225,8 @@ <h2 style="margin: 0;">Workout tools</h2>
const recordWeight = document.getElementById('record-weight');
const recordReps = document.getElementById('record-reps');
const recordsCaption = document.getElementById('records-caption');
const recordsContext = document.getElementById('records-context');
const noRecordNote = document.getElementById('no-record-note');

const STORAGE_KEYS = { base: 'workout-api-base', token: 'workout-api-token' };

Expand Down Expand Up @@ -314,9 +323,13 @@ <h2 style="margin: 0;">Workout tools</h2>
function updateRecordsCaption(name, hasRecords) {
if (!name) {
recordsCaption.textContent = 'Select an exercise to view records';
recordsContext.textContent = 'No exercise selected.';
noRecordNote.textContent = '';
return;
}

recordsContext.textContent = `Showing records for: ${name}`;

recordsCaption.textContent = hasRecords === false
? `No ${name} records`
: `${name} records`;
Expand All @@ -333,6 +346,7 @@ <h2 style="margin: 0;">Workout tools</h2>
updateRecordsCaption(displayName);
loadStatus.textContent = `Loading records for ${displayName}...`;
recordRows.innerHTML = '';
noRecordNote.textContent = '';
try {
const data = await apiFetch(`/exercises/${encodeURIComponent(name)}/records`);
(data.records || []).forEach((rec) => {
Expand All @@ -344,11 +358,13 @@ <h2 style="margin: 0;">Workout tools</h2>
const row = document.createElement('tr');
row.innerHTML = '<td colspan="2">No personal bests yet.</td>';
recordRows.appendChild(row);
noRecordNote.textContent = `${displayName} has no records yet.`;
}
updateRecordsCaption(displayName, data.records && data.records.length > 0);
loadStatus.textContent = '';
} catch (error) {
loadStatus.textContent = error.message;
noRecordNote.textContent = '';
}
}

Expand All @@ -374,17 +390,23 @@ <h2 style="margin: 0;">Workout tools</h2>
exerciseSelect.addEventListener('input', () => {
clearRecordInputs();
recordStatus.textContent = '';
loadStatus.textContent = '';
updateRecordsCaption('');
recordRows.innerHTML = '';
});
if (!getSelectedExercise()) {
loadStatus.textContent = '';
updateRecordsCaption('');
recordRows.innerHTML = '';
return;
}

loadRecords();
});
exerciseBrowseSelect.addEventListener('change', (event) => {
const selectedOption = event.target.selectedOptions[0];
if (!selectedOption) return;
const value = selectedOption.value || selectedOption.textContent;
exerciseSelect.value = value;
exerciseSelect.dispatchEvent(new Event('input', { bubbles: true }));
clearRecordInputs();
recordStatus.textContent = '';
loadRecords();
const details = exerciseBrowseSelect.closest('details');
if (details) details.open = false;
});
Expand Down