Feature request: respond to KEYCODE_MEDIA* media buttons

[sudomain]

Is your feature request related to a problem? Please describe.

Please have the app respond to media buttons such as
KEYCODE_MEDIA_STOP, KEYCODE_MEDIA_PLAY, and KEYCODE_MEDIA_PLAY_PAUSE so that sounds can be affected by hardware media buttons.
https://developer.android.com/guide/topics/media-apps/mediabuttons

Describe the solution you'd like

Play, pause, stop, and play/pause toggle via media buttons please. Another idea could be KEYCODE_MEDIA_NEXT and KEYCODE_MEDIA_PREVIOUS to cycle through the saved presets.

[ashutoshgngwr]

And you'd think that media session callbacks will handle that for you. I am really starting to hate how Android is turning out to be lately.

https://github.com/ashutoshgngwr/noice/blob/8da55e749f37b985110dcd5eecbc7132ae0aa356/app/src/main/java/com/github/ashutoshgngwr/noice/sound/player/PlayerManager.kt#L75-L91

---

Edit

What exactly is the purpose of Media Session callbacks then? It doesn't seem to handle media buttons.

https://developer.android.com/reference/kotlin/android/support/v4/media/session/MediaSessionCompat.Callback

---

Wow! I am bamboozled.

https://developer.android.com/reference/android/support/v4/media/session/MediaSessionCompat.Callback#onMediaButtonEvent(android.content.Intent)

---

Found the issue. Mistakenly used bitwise AND instead of OR. Right now, I have a lot of uncommitted changes in my working tree. Will fix this ASAP.

https://github.com/ashutoshgngwr/noice/blob/8da55e749f37b985110dcd5eecbc7132ae0aa356/app/src/main/java/com/github/ashutoshgngwr/noice/sound/player/PlayerManager.kt#L68-L73

[sudomain]

Thank you for the quick response on this and the other issue. What is your opinion on KEYCODE_MEDIA_NEXT and KEYCODE_MEDIA_PREVIOUS to cycle through the saved presets?

[ashutoshgngwr]

@sudomain I'll be working on that among other things. I was going to add media player style notifications, but that requires some code restructuring before implementation. These media buttons should be partially implemented during the notification implementation. Nevertheless, it'll all be available in the next release.