Skip to content

Conversation

@mathiash98
Copy link
Contributor

@mathiash98 mathiash98 commented Apr 24, 2016

As requested by another user. issue #716
Done using triggerHandler on ".player_favorite"

Done using triggerHandler on ".player_favorite"
@weblancaster weblancaster changed the title Added hotkey (ctrl+l) to like/favorite track Added hotkey (ctrl+l) to like/favorite track. closes #716 Apr 26, 2016
@zedeus
Copy link

zedeus commented May 18, 2016

Currently, the shortcut CTRL+L "locks" the track, you may wanna look into that

@weblancaster
Copy link
Member

@zestyr is right @mathiash98 you will need to change that

@zedeus
Copy link

zedeus commented May 18, 2016

I'd suggest Ctrl+H, since it isn't taken, and it's H as in Heart

@mathiash98
Copy link
Contributor Author

@zestyr @weblancaster edited to ctrl+H now

@mathiash98 mathiash98 changed the title Added hotkey (ctrl+l) to like/favorite track. closes #716 Added hotkey EDIT: (ctrl+H) to like/favorite track. closes #716 May 19, 2016
hotkeys.add({
combo: ['command+h', 'ctrl+h'],
description: 'Heart track',
callback: function() {
Copy link
Member

Choose a reason for hiding this comment

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

why the $timeout here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have no idea, I just copied your code from another function. @weblancaster

Copy link
Member

Choose a reason for hiding this comment

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

Hm I checked not sure why we have that timeout over there.. please remove and test again if works just leave the $timeout out and I will merge.. thanks

@ghost
Copy link

ghost commented Oct 8, 2016

@mathiash98 agreed, this is implementation is very important.

@weblancaster weblancaster merged commit 7cf30b2 into Soundnode:master Oct 13, 2016
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.

3 participants