-
Notifications
You must be signed in to change notification settings - Fork 640
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
Added hotkey EDIT: (ctrl+H) to like/favorite track. closes #716 #752
Conversation
Done using triggerHandler on ".player_favorite"
Currently, the shortcut CTRL+L "locks" the track, you may wanna look into that |
@zestyr is right @mathiash98 you will need to change that |
I'd suggest Ctrl+H, since it isn't taken, and it's H as in Heart |
@zestyr @weblancaster edited to ctrl+H now |
combo: ['command+h', 'ctrl+h'], | ||
description: 'Heart track', | ||
callback: function() { | ||
$timeout(function() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why the $timeout
here?
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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
@mathiash98 agreed, this is implementation is very important. |
As requested by another user. issue #716
Done using triggerHandler on ".player_favorite"