Skip to content
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.

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

[podcast] Stop Audio Playback on Windows and Linux #84

Merged
merged 1 commit into from Nov 29, 2023

Conversation

ricoberger
Copy link
Member

On Windows and Linux it could happen that the audio playback for a podcast wasn't stopped when the item details view for a podcast item was closed.

This commit "fixes" the problem, by pausing the audio player, before the widget is disposed.

This commit also fixes the condition when the background audio services should be initialized in the main.dart file. Instead of the macOS check, we checked for iOS twice.

On Windows and Linux it could happen that the audio playback for a
podcast wasn't stopped when the item details view for a podcast item was
closed.

This commit "fixes" the problem, by pausing the audio player, before the
widget is disposed.

This commit also fixes the condition when the background audio services
should be initialized in the `main.dart` file. Instead of the macOS
check, we checked for iOS twice.
@ricoberger ricoberger added the changelog: fixed Something which wasn't working is fixed label Nov 29, 2023
@ricoberger ricoberger self-assigned this Nov 29, 2023
@ricoberger ricoberger added this to the v1.3.0 milestone Nov 29, 2023
@ricoberger ricoberger merged commit a58c93b into main Nov 29, 2023
8 checks passed
@ricoberger ricoberger deleted the podcast-stop-audio-playback-on-windows-and-linux branch November 29, 2023 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog: fixed Something which wasn't working is fixed
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

1 participant