CalvinBackup/SpotiFLAC-Mobile
1
0
Fork 0
mirror of https://github.com/zarzet/SpotiFLAC-Mobile.git synced 2026-07-25 21:40:57 +02:00
Code Issues Packages Projects Releases Wiki Activity

fix(queue): keep just-completed tracks pinned while the batch is running

Browse Source 
A track that finished downloading dropped out of the active lead zone
and re-appeared in the history zone below every still-queued sibling,
visually jumping several positions during album downloads. The existing
completion bridge masked this for only 6 seconds.

Keep completion-bridge cells pinned in the lead zone for as long as
other downloads are still active, and hide their history rows while
pinned so tracks don't render twice. Once the queue goes idle the
bridge dismisses and items settle into their normal history position.
This commit is contained in:
zarzet
2026-07-09 19:47:13 +07:00
parent 60bf871945
commit bb66cc470a
1 changed files with 44 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/screens/queue_tab.dart
+44 -22
View File
@@ -4200,28 +4200,9 @@ class _QueueTabState extends ConsumerState<QueueTab> {
final filteredGroupedAlbums = filterData.filteredGroupedAlbums;
final filteredGroupedLocalAlbums = filterData.filteredGroupedLocalAlbums;
final unifiedItems = filterData.unifiedItems;
final filteredUnifiedItems = filterData.filteredUnifiedItems;
final allFilteredUnifiedItems = filterData.filteredUnifiedItems;
final totalTrackCount = filterData.totalTrackCount;
final totalAlbumCount = filterData.totalAlbumCount;
final downloadedNavigationItems = <DownloadHistoryItem>[];
final downloadedNavigationIndexByUnifiedId = <String, int>{};
final localNavigationItems = <LocalLibraryItem>[];
final localNavigationIndexByUnifiedId = <String, int>{};
for (final item in filteredUnifiedItems) {
final historyItem = item.historyItem;
if (historyItem != null) {
downloadedNavigationIndexByUnifiedId[item.id] =
downloadedNavigationItems.length;
downloadedNavigationItems.add(historyItem);
}
final localItem = item.localItem;
if (localItem != null) {
localNavigationIndexByUnifiedId[item.id] = localNavigationItems.length;
localNavigationItems.add(localItem);
}
}
final activeDownloadIds = filterMode == 'albums'
? const <String>[]
@@ -4243,18 +4224,29 @@ class _QueueTabState extends ConsumerState<QueueTab> {
.reversed
.toList(growable: false);
final libIdSet = <String>{for (final item in filteredUnifiedItems) item.id};
final libIdSet = <String>{
for (final item in allFilteredUnifiedItems) item.id,
};
List<String> bridgeIds = const [];
if (filterMode != 'albums' && _completionBridge.isNotEmpty) {
final now = DateTime.now();
final stale = <String>[];
final pending = <String>[];
final hasActiveDownloads = activeDownloadIds.isNotEmpty;
_completionBridge.forEach((id, _) {
final landed = libIdSet.contains('dl_$id');
final addedAt = _completionBridgeAt[id];
final expired =
addedAt == null || now.difference(addedAt).inSeconds >= 6;
if (landed || expired || activeDownloadIds.contains(id)) {
if (activeDownloadIds.contains(id)) {
// Re-queued (retry) — the live row takes over from the bridge.
stale.add(id);
} else if (hasActiveDownloads) {
// Keep just-completed tracks pinned in the lead zone while the
// rest of the batch is still downloading, so they don't jump
// below the remaining queue the moment they finish.
pending.add(id);
} else if (landed || expired) {
stale.add(id);
} else {
pending.add(id);
@@ -4313,6 +4305,36 @@ class _QueueTabState extends ConsumerState<QueueTab> {
}
}
// Tracks pinned as completion-bridge cells render in the lead zone;
// hide their history rows so they don't appear twice.
List<UnifiedLibraryItem> filteredUnifiedItems = allFilteredUnifiedItems;
if (bridgeIds.isNotEmpty) {
final pinnedHistoryIds = <String>{for (final id in bridgeIds) 'dl_$id'};
filteredUnifiedItems = allFilteredUnifiedItems
.where((item) => !pinnedHistoryIds.contains(item.id))
.toList(growable: false);
}
final downloadedNavigationItems = <DownloadHistoryItem>[];
final downloadedNavigationIndexByUnifiedId = <String, int>{};
final localNavigationItems = <LocalLibraryItem>[];
final localNavigationIndexByUnifiedId = <String, int>{};
for (final item in filteredUnifiedItems) {
final historyItem = item.historyItem;
if (historyItem != null) {
downloadedNavigationIndexByUnifiedId[item.id] =
downloadedNavigationItems.length;
downloadedNavigationItems.add(historyItem);
}
final localItem = item.localItem;
if (localItem != null) {
localNavigationIndexByUnifiedId[item.id] = localNavigationItems.length;
localNavigationItems.add(localItem);
}
}
final leadCount = activeDownloadIds.length + bridgeIds.length;
final collectionEntries = filterMode == 'all'
? _getVisibleCollectionEntries(collectionState)