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

Custom Keyboard Shortcuts #40

Closed
killa-kyle opened this issue Dec 16, 2015 · 9 comments
Closed

Custom Keyboard Shortcuts #40

killa-kyle opened this issue Dec 16, 2015 · 9 comments
Labels

Comments

@killa-kyle
Copy link
Contributor

It would be nice to be able to "Like" the current playing song with a keyboard shortcut.
I love the ability to go through my Soundcloud feed with the keyboard media buttons, and being able to like tracks as I hear them would be useful.

@gillesdemey
Copy link
Owner

I've contemplated this feature as well, unfortunately Electron's global shortcuts take over any other similar shortcut defined across your system so we'll have to some up with something pretty unique for Cumulus. Feel free to make a suggestion! :)

@andersfischernielsen
Copy link

Cmd+Alt+L doesn't seem to be used very often by apps on OS X. :)

@frdmn
Copy link

frdmn commented Apr 15, 2016

👍

@killa-kyle
Copy link
Contributor Author

killa-kyle commented May 21, 2016

I submitted a pull request for this (#61). Went with @andersfischernielsen 's Cmd+Alt+L and it works well for me ¯_(ツ)_/¯

@seppestas
Copy link
Collaborator

This was fixed in #61

@FoxxMD
Copy link

FoxxMD commented Nov 16, 2016

Cmd+Alt+L is used by Intellij to reformat code. Took me forever to figure out Cumulus was hijacking it :/

@gillesdemey
Copy link
Owner

@FoxxMD I'm sorry this is happening to you, I hope this hasn't impacted your workflow too much.

I propose to add this behaviour as a setting that you have to opt-in to.

@seppestas
Copy link
Collaborator

To bad OSes (or at least macOS) don't have a way to centralize shortcuts for applications.

Imagine an API applications have to use to register all shortcuts and the scope in which they are used. This would allow the the OS to detect collisions and tell the user. Please, someone make this happen!

@FoxxMD
Copy link

FoxxMD commented Nov 16, 2016

@seppestas in a perfect world!

@gillesdemey It's no problem since I figured it out!

Opt-in would definitely be a better behavior.

Or document it in an obvious place inside the app or on the homepage. I shouldn't have to dig through github issues to find out there's a global shortcut that overrides everything else in my system ;) If I was a normal user (read: not a developer myself and unfamiliar with github) I'd be tearing my hair out by now.

Edit: but then again I wouldn't be using intellij and not be a dev....but you get the point!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

7 participants