Why don’t we allow shortcuts with Shift and Alt modifiers?

[zoul]

Recently I have found that MASShortcut doesn’t record shortcuts such as Shift-Alt-U. The reason is in the validation logic in MASShortcutValidator. I can understand why we disallow Alt-something, but shouldn’t Shift-Alt-something work? @shpakovski, do you have an idea why Shift-Alt-something is not considered valid? Thanks!

[shpakovski]

@zoul I did not want to encourage these shortcuts because Shift-Option works just like Option. For example, Shift-Option-K = .

UPD There is a property allowAnyShortcutWithOptionModifier by the way, have you enabled it?

[zoul]

Makes sense, thank you! (I will document the behaviour so that people know the thinking behind it.) I noticed the allowAny… switch, but I wasn’t sure what the exact deal was, what were the potential drawbacks, so I didn’t touch it. I’ll leave the issue open as a reminder to update the docs.

[shpakovski]

Cool, thanks!

[SenorSamuel]

that's really cool