Skip to content

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

Open
joegege wants to merge 2 commits into
fvacek:masterfrom
joegege:master
Open

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

Conversation

@joegege

@joegege joegege commented Jul 6, 2013

Copy link
Copy Markdown
Contributor

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

fvacek commented Jul 6, 2013

Copy link
Copy Markdown
Owner

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

fvacek commented Jul 6, 2013

Copy link
Copy Markdown
Owner

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

joegege commented Jul 7, 2013

Copy link
Copy Markdown
Contributor Author

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