diff options
| author | Paul Buetow <paul@buetow.org> | 2026-05-22 09:52:26 +0300 |
|---|---|---|
| committer | Paul Buetow <paul@buetow.org> | 2026-05-22 09:52:26 +0300 |
| commit | b0f07dd0a4535a330553e323417ccdd354fae22f (patch) | |
| tree | a39fb53a0dc6206d57c690ea11d317b9a20772cc /player-android/lib/providers/progress_queue_provider.dart | |
| parent | aaf61a9e3c0e1439faee36c21462e0788639b7f1 (diff) | |
Fix bb review issues: stale snapshot, double-tap guard, button dedup, comments
- _downloadEpisodeAt: read _episodes fresh inside setState instead of using
the pre-await snapshot, and guard that the index is still valid and the row
still lacks a mediaId, preventing silent overwrites of data refreshed by
_load() during the await.
- Add Set<int> _pendingDownloads to prevent concurrent download API calls when
the user double-taps; visually disable the button while in-flight via new
isLoading parameter on _DownloadButton.
- Extract _EpisodeActionButton shared primitive to eliminate structural
duplication between _PlayButton and _DownloadButton (DRY).
- Fix misleading mediaId null-safety comment to describe the actual guarantee.
- Add comments to episodeToggleErrorMessage and episodeDownloadErrorMessage
explaining why they use action-specific fallback strings.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Diffstat (limited to 'player-android/lib/providers/progress_queue_provider.dart')
0 files changed, 0 insertions, 0 deletions
