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

Use MPNowPlayingInfoCenter to player info #77

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

aussiegeek
Copy link
Contributor

This means that the notications bar media player will show track &
artist title, as well as artwork for the album.

It also means that Play/Pause/Next/Previous work without needing
accessibility access

This means that the notications bar media player will show track &
artist title, as well as artwork for the album.

It also means that Play/Pause/Next/Previous work without needing
accessibility access
@aussiegeek
Copy link
Contributor Author

Note that I'm fairly new to Swift, and have never used ReSwift, so realise there could be a much better way of doing this


class PlayerStateInfoController {
init() {
App.store.subscribe(self) {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We subscribe to the current song separately from the play/pause status so we don't continually update the current album, including fetching artwork

Base automatically changed from master to main March 13, 2021 15:39
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.

None yet

1 participant