cocoabits / MASShortcut

Modern framework for managing global keyboard shortcuts compatible with Mac App Store. More details:
http://blog.shpakovski.com/2012/07/global-keyboard-shortcuts-in-cocoa.html
BSD 2-Clause "Simplified" License
1.52k stars 220 forks source link

Overwriting shortcut warning only partially working #142

Open dataexcess opened 4 years ago

dataexcess commented 4 years ago

Hello,

First of thanks for the awesome library. I am experiencing issues when trying to set a shortcut for my MASShortcutView.

When I try to set "command-n" it correctly displays the following warning: This shortcut cannot be used because it is already used by the menu item ‘New’. When I try "command-shift-n" it does not complain, and the shortcut is set. But that combination is also used by Finder, why did it not detect that? The same goes for other shortcuts, such as "alt-command-L" etc.

Thank you for clarifying.

All the best, J

Kentzo commented 4 years ago

Neither shortcut is global: they are handled only when the Finder application is active. It's impossible (without employing intrusive malware-alike actions) to determine shortcuts used by other apps.

shpakovski commented 4 years ago

Thanks @Kentzo 👍

dataexcess commented 4 years ago

Ok @Kentzo thank you for clarifying. I guess the command+n shortcut overwrite was originating from the hidden menu in my own application and I mistook it for finder's command.