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

[osx] fix media key control for next and previous track #10182

Merged
merged 1 commit into from
Jul 31, 2016

Conversation

Rechi
Copy link
Member

@Rechi Rechi commented Jul 29, 2016

This allows to use the media controls for previous and next track on mac.
Without this PR they would invoke f14 and f15 which have no action set in the keymaps.

@MartijnKaijser
Copy link
Member

@Memphiz ping

@Memphiz
Copy link
Member

Memphiz commented Jul 29, 2016

mhhh i wonder why those had wrong keycodes then before @davilla do you have a clue about that? (unlikely i know...) - well lets hope that its not something that only works with newer osx versions then ... jenkins build this please

@MrMC
Copy link

MrMC commented Jul 29, 2016

not a clue, too far back in time.

@@ -442,8 +442,8 @@ - (void) deviceDidUnMountNotification:(NSNotification *) note
#define VK_MEDIA_PREV_TRACK 0xB1
#define VK_MEDIA_STOP 0xB2
#define VK_MEDIA_PLAY_PAUSE 0xB3
#define VK_REWIND 0x9D
#define VK_FAST_FWD 0x9E
#define VK_REWIND 0xB1

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

@Rechi
Copy link
Member Author

Rechi commented Jul 31, 2016

@Memphiz swapped the codes

@Memphiz
Copy link
Member

Memphiz commented Jul 31, 2016

I'm ok with this. Please have a look in the osx forum in case there are issues with this reported by users - if you can .

@Memphiz Memphiz merged commit d03b0a9 into xbmc:master Jul 31, 2016
@Rechi Rechi deleted the osxMediaControl branch July 31, 2016 13:50
@MartijnKaijser MartijnKaijser added this to the Krypton 17.0-alpha3 milestone Jul 31, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants