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

Added hotkey EDIT: (ctrl+H) to like/favorite track. closes #716 #752

Merged
merged 2 commits into from
Oct 13, 2016
Merged

Added hotkey EDIT: (ctrl+H) to like/favorite track. closes #716 #752

merged 2 commits into from
Oct 13, 2016

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
combo: ['command+h', 'ctrl+h'],
description: 'Heart track',
callback: function() {
$timeout(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