quoije/oscars-pool
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Refactor player UI settings to combine source and progress

Browse Source 
Co-authored-by: mathieumoran1 <[email protected]>
This commit is contained in:
Cursor Agent
2025-12-22 21:10:07 +00:00
co-authored by mathieumoran1
parent a227184982
commit 420d26dfaa
3 changed files with 37 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files
public/js/app_control.js
+8 -8
View File
@@ -42,8 +42,7 @@ window.onload = async function () {
const saveActiveYearBtn = document.getElementById('save-active-year');
// Player UI (settings tab): show/hide admin status blocks in player.html
const playerUiShowSourceEl = document.getElementById('player_ui_show_source');
const playerUiShowProgressEl = document.getElementById('player_ui_show_progress');
const playerUiShowStatusEl = document.getElementById('player_ui_show_status');
const playerUiReloadBtn = document.getElementById('player_ui_reload');
const playerUiSaveBtn = document.getElementById('player_ui_save');
@@ -193,21 +192,22 @@ window.onload = async function () {
}
function setPlayerUiForm(value) {
if (!playerUiShowSourceEl || !playerUiShowProgressEl) return;
playerUiShowSourceEl.checked = !!value.showSource;
playerUiShowProgressEl.checked = !!value.showProgress;
if (!playerUiShowStatusEl) return;
// Single checkbox controls both flags.
playerUiShowStatusEl.checked = !!value.showSource && !!value.showProgress;
}
function getPlayerUiFormValue() {
const enabled = !!playerUiShowStatusEl?.checked;
return normalizePlayerUi({
showSource: !!playerUiShowSourceEl?.checked,
showProgress: !!playerUiShowProgressEl?.checked,
showSource: enabled,
showProgress: enabled,
});
}
async function loadPlayerUiIntoUi() {
// If the section isn't present (older HTML), skip gracefully.
if (!playerUiShowSourceEl) return;
if (!playerUiShowStatusEl) return;
try {
const value = await fetchPlayerUi();
setPlayerUiForm(value);