Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Thor shortcut #47

Open
jvansant opened this issue Jan 17, 2022 · 3 comments · May be fixed by #67
Open

Thor shortcut #47

jvansant opened this issue Jan 17, 2022 · 3 comments · May be fixed by #67

Comments

@jvansant
Copy link

The "Enable shortcut" toggle in the Thor settings seems to do nothing. I would prefer to use shift control opt cmd + T for a different application. Currently no matter how I configure the Thor settings this shortcut is not able to be assigned to another application.

@kenji21
Copy link
Contributor

kenji21 commented Jan 18, 2022

I think you misunderstood what the shortcut does:

Screenshot 3

@fuzzyweapon
Copy link

Although they misunderstood the Enable shortcut option, this was confusing to me too and should probably be clarified, such as, Enable Custom Shortcuts

However, I think their issue is still value. I understand what they are saying. Thor will hog up that key binding slot (for hiding and showing the tray icon) globally and there is no way to free it up. What they want is the freedom to not have Thor take up that binding.

I can relate to this as an IDE user. This is very problematic and there is no shortcut to be able to easily turn Thor shortcuts on and off (a toggle, not something that re-enables itself so quickly; though this feature is cool, it doesn't allow me ever enough time to do what I need, even at 1s).

Besides, using this shortcut only enables me to have to use my mouse to interact with Thor which feels weird. It doesn't feel very high value compared to have this binding forcefully eaten up by Thor when I can use an existing shortcut to minimize it and use my mouse then to restore it from the dock that way anyways.

@wiiwaker
Copy link

I second this issue and hope it will be implemented soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants