danvics/soulsync
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

WebUI: Quality Profile tile visible via tab filter (drop fragile Soulseek gate)

Browse source 
The Soulseek-only JS gate fought the settings tab filter for control of the
tile's display: gating the inner group left an empty expandable shell, gating
the wrapper hid the whole tile depending on activeSources/tab timing. Remove
the JS gate entirely and let the tab filter (data-stg="downloads" on
#quality-profile-tile) own visibility — identical to the working Retry Logic
tile. The tile now reliably shows on the Downloads tab.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
dev 2026-06-05 22:26:35 +02:00
parent 81291c198b
commit 87a4e41f9e
1 changed files with 7 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
webui/static/settings.js
View file

@ -1711,15 +1711,13 @@ function updateDownloadSourceUI() {
prowlarrRedirect.style.display = showProwlarr ? 'block' : 'none';
}
// Quality profile is Soulseek-only and downloads-tab-only. Gate the whole
// collapsible tile (header + body) — gating just the inner #quality-profile-section
// left the tile header expandable to an empty body.
const qualityProfileTile = document.getElementById('quality-profile-tile');
if (qualityProfileTile) {
const activeTab = document.querySelector('.stg-tab.active');
const onDownloadsTab = activeTab && activeTab.dataset.tab === 'downloads';
qualityProfileTile.style.display = (activeSources.has('soulseek') && onDownloadsTab) ? '' : 'none';
}
// Quality Profile tile visibility is handled entirely by the settings tab
// filter (data-stg="downloads" on #quality-profile-tile) — the same mechanism
// the Retry Logic tile uses. The old Soulseek-only JS gate fought the tab
// filter for control of `display` and, depending on activeSources/tab timing,
// left the tile either empty (when it gated the inner group) or fully hidden
// (when it gated the wrapper). Letting the tab filter own visibility makes it
// behave exactly like the working Retry Logic tile.
if (activeSources.has('tidal')) {
checkTidalDownloadAuthStatus();