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

78-media-playback-multiplatform #79

Merged
merged 6 commits into from
Jan 6, 2025
Merged

Conversation

odifek
Copy link
Member

@odifek odifek commented Jan 6, 2025

  • implement BottomControlsUi and sample audio playback
  • implement custom back arrow
  • fix detail screen flashing in dark mode
  • add logic to import bundled tunes archive and save them in the database

The the rest of the commits

odifek added 5 commits January 5, 2025 09:09
implement custom back arrow

fix detail screen flashing in dark mode
…ile isPlaying is true.

mediaId parameter is added to the AudioItem model. This will be used also to open the detail screen directly from the notification.
The idea is to format the media id like songbook||entry. Then we can open to the song entry directly. TBD
We use the mediaId also to determine when the current page has changed.

add DummyPlaybackController for android composable preview
add logic to navigate using the previous/next buttons, show songbook and number on the bottom controls
Fixes the issue where multiple controllers could be active at the same time.

Similar to AndroidPlaybackController but only that playback is also stopped.
Future improvement: continue the playback in the background. Look into iOS audio session
@odifek odifek self-assigned this Jan 6, 2025
@odifek odifek linked an issue Jan 6, 2025 that may be closed by this pull request
storing absolute path breaks the app on iOS as each update is installed in a different directory
@odifek odifek force-pushed the 78-media-playback-multiplatform branch from fe927c6 to 660a21c Compare January 6, 2025 00:32
@odifek odifek merged commit 164adf9 into master Jan 6, 2025
1 check failed
@odifek odifek deleted the 78-media-playback-multiplatform branch January 6, 2025 00:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

media playback multiplatform
1 participant