Skip to content

✨ Allow select keybinds to ignore the trigger key#963

Merged
mrkai77 merged 9 commits into
mrkai77:developfrom
cipher-shad0w:issue_260
Jan 26, 2026
Merged

✨ Allow select keybinds to ignore the trigger key#963
mrkai77 merged 9 commits into
mrkai77:developfrom
cipher-shad0w:issue_260

Conversation

@cipher-shad0w

Copy link
Copy Markdown
Contributor

@cipher-shad0w

Copy link
Copy Markdown
Contributor Author

There’s just one problem right now — currently, you can set any single key as a trigger for an action, for example just a single letter. What should I check for? A minimum of 2 keys? And certain keys that must be included so that the custom trigger key can be used?

@mrkai77 mrkai77 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There’s just one problem right now — currently, you can set any single key as a trigger for an action, for example just a single letter. What should I check for? A minimum of 2 keys? And certain keys that must be included so that the custom trigger key can be used?

I'd say a minimum of at least one modifier key and one non-modifier key, and then a maximum of around 5 keys in total (e.g. 4 modifiers, one key).

Comment thread Loop/Window Management/Window Action/WindowAction.swift Outdated
@cipher-shad0w

Copy link
Copy Markdown
Contributor Author

I'd say a minimum of at least one modifier key and one non-modifier key, and then a maximum of around 5 keys in total (e.g. 4 modifiers, one key).

Does Loop has anything implemented to check if sth is a modifier key or not?

@mrkai77

mrkai77 commented Jan 7, 2026

Copy link
Copy Markdown
Owner

There is CGKeyCode.isModifier. Additionally, it likely makes sense to map these via CGKeyCode.baseModifier to eliminate the left/right key code distinction.

@mrkai77 mrkai77 added the Feature label Jan 7, 2026
@mrkai77 mrkai77 marked this pull request as draft January 25, 2026 04:16
@mrkai77

mrkai77 commented Jan 25, 2026

Copy link
Copy Markdown
Owner

@cipher-shad0w do you have any updates on this?

@cipher-shad0w

Copy link
Copy Markdown
Contributor Author

Oh yeah sry, I've implemented everything you ask for and fixed everything you commented

@mrkai77 mrkai77 marked this pull request as ready for review January 26, 2026 03:34

@mrkai77 mrkai77 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I made a few changes that fixed some bugs on my end and also optimized parts of both your code and the existing code. Everything works perfectly now, thank you!

@mrkai77 mrkai77 merged commit 6a06286 into mrkai77:develop Jan 26, 2026
1 check passed
@mrkai77 mrkai77 changed the title ✨ Add generic keyboard shortcut to trigger Loop ✨ Allow select keybinds to ignore the trigger key Jan 26, 2026
@cipher-shad0w cipher-shad0w deleted the issue_260 branch January 26, 2026 20:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants