Skip to content

Supports media keys: long press volume up , long press volume down, play/pasue.#1

Open
joegege wants to merge 2 commits intofvacek:masterfrom
joegege:master
Open

Supports media keys: long press volume up , long press volume down, play/pasue.#1
joegege wants to merge 2 commits intofvacek:masterfrom
joegege:master

Conversation

@joegege
Copy link
Copy Markdown
Contributor

@joegege joegege commented Jul 6, 2013

long press volume Up - next song;
long press volume Down - previous song(not work actually, because long press consume 600+ms);
play/pause key;
x - quit application;

For long press volume down, can only back to the begin of current song, because long press consume 600+ms.

joegege added 2 commits July 6, 2013 09:00
long press volume Up - next song;
long press volume Down - previous song(not work actually, because long press consume 600+ms);
play/pause key;
x - quit application;
@fvacek
Copy link
Copy Markdown
Owner

fvacek commented Jul 6, 2013

I think, that I can merge your changes to master branch, when review phase will be finished. Please look at my comments on your pull request.

@fvacek
Copy link
Copy Markdown
Owner

fvacek commented Jul 6, 2013

ad For long press volume down, can only back to the begin of current song, because long press consume 600+ms.:

I think that we can change to previous track threshold to let say 650ms

@joegege
Copy link
Copy Markdown
Contributor Author

joegege commented Jul 7, 2013

I have refined code and submit new pull request.
Please ignore this one.
Thanks.

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.

2 participants