Hard-coded keyboard shortcuts

[sindresorhus]

Sometimes you need to add hard-coded keyboard shortcuts. Would be useful to add an API for that.

It could be an overload to onKeyUp/onKeyDown that accepts a Key and NSEvent.ModifierFlags.

KeyboardShortcuts.onKeyUp(.a, modifiers: [.command]) {
	// Do something.
}

Alternatively, it could just support a KeyboardShortcuts.Shortcut type directly:

KeyboardShortcuts.onKeyUp(.init(.a, modifiers: [.command])) {
	// Do something.
}

We should document that devs should prefer to let users customize the shortcuts instead.

Feedback wanted.

[jameshfisher]

(First off, thanks for this library - it looks great!)

I believe this issue is describing what I was looking for. In my app, I need to be able to say:

To pause or unpause, press Fn+Ctrl+p. If you prefer a different shortcut, you can customize it in Preferences.

I can't see a way to implement that with the current KeyboardShortcuts API. It seems to require the user to provide a custom Shortcut for a Name before it can be used.

I can see that the API lets me construct a KeyboardShortcuts.Shortcut, e.g.

KeyboardShortcuts.Shortcut(.p, modifiers: [.function, .control])

However, I can't do anything with this shortcut, so I'm not sure why it's available in the API at the moment.

I'm considering hacking around this by reading and writing to UserDefaults directly, but that seems ugly and brittle.

I'm happy to work on the implementation, assuming I've understood correctly so far.

[jameshfisher]

Aha, 5 minutes later, I stumbled on this merged PR which does exactly what I needed: #13

It's not documented, though. I can update the docs, if you wish.

[sindresorhus]

It's not documented, though. I can update the docs, if you wish.

It is documented: https://sindresorhus.com/KeyboardShortcuts/Enums/KeyboardShortcuts/Name.html#/s:17KeyboardShortcutsAAO4NameV_7defaultADSS_AB8ShortcutVSgtcfc

[sindresorhus]

To pause or unpause, press Fn+Ctrl+p. If you prefer a different shortcut, you can customize it in Preferences.

Keep this in mind though:

Users find it annoying when random apps steal their existing keyboard shortcuts. It’s generally better to show a welcome screen on the first app launch that lets the user set the shortcut.

[jameshfisher]

@sindresorhus ah, yes, I meant the README, which is the main place I was looking 😳

The warning makes sense. In my case, I'm migrating existing behavior, and the app is session-based rather than a long-lived menu bar app, which I think makes it less annoying.

[Vincz]

Hi @sindresorhus!
Any update about this? The library is amazing, but I'd like to use it without the UserDefaults stuff.
I think the new API you suggest would solve a large part of what I want to do.

Thanks!