Fix: filter out shortcut triggers where key-modifier is same

[AnkushJadhav]

Description:

Shortcuts triggered an event to the queue for the shortcut's modifier key press. Added a filter to prevent the modifier key press from triggering the shortcut callback.

Fixes #2148

Checklist:

• Tests included.
• Lint and formatter run with no errors.
• Tests all pass.

[andydotxyz]

This is a great fix thanks, would it be possible to add a test that shows the shortcut is no longer sent to an Entry?

[andydotxyz]

Oh one other thought - we try to keep method names in alphabetical order (grouped by type) so the new method probably belongs further down the file.

[AnkushJadhav]

@andydotxyz Thanks. Will make the changes and update the PR asap. Since the fix is applicable for any component on the window that implements TypedShortcut, should the test for this be on window or on the entry widget?

[AnkushJadhav]

@andydotxyz Added tests to a generic implementation of Shortcutable instead of entry, as the code change impacts all implementations of Shortcutable

[andydotxyz]

Great thank you very much :)