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

[bug] keybindings override system shortcuts #211

Open
roloit opened this issue Oct 19, 2024 · 6 comments
Open

[bug] keybindings override system shortcuts #211

roloit opened this issue Oct 19, 2024 · 6 comments
Assignees
Labels

Comments

@roloit
Copy link

roloit commented Oct 19, 2024

Weird thing now, probably a bug? On macOS, if I click play, then hide the window, go to another application/tab, and try to force quit the application (Say I'm in a browser, decide to force quit it with shortcut, muffon is hidden), instead of the application I'm in to quit, muffon quits, and it happens even if I don't have any windows open... It happens in version 2.1.0, version 2.0.3 doesn't have that issue.

@xyloflake
Copy link
Contributor

@roloit can you attach a video of the bug? I can't reproduce it on my end (macos Monterrey).

@roloit
Copy link
Author

roloit commented Oct 19, 2024

So first I start playing something, hide the window, then click the force quit shortcut while the application is hidden, it quits the application, it's very annoying when you try to quit other apps, and first, muffon quits, only then you can quit the application you're in. macOS Sequioa
https://github.com/user-attachments/assets/e18b2e4e-8114-4e89-879a-b5efb13ce311

@xyloflake
Copy link
Contributor

So first I start playing something, hide the window, then click the force quit shortcut while the application is hidden, it quits the application, it's very annoying when you try to quit other apps, and first, muffon quits, only then you can quit the application you're in. macOS Sequioa https://github.com/user-attachments/assets/e18b2e4e-8114-4e89-879a-b5efb13ce311

Hmm.... I'll check the code for just-in-case.

For confirmation @roloit : you're holding down Cmd+Q right? Also please check the menu bar, on the top left, it should show the application name you're focussed on, if muffon is in focus, it will force quit.

@xyloflake
Copy link
Contributor

Found the bug, no need to worry. Thank you for the report. I'll go ahead and report this.

@roloit this will be fixed in the next version. We apologize for the inconvenience caused.

globalShortcut.register(

@staniel359 @gamersi this registers a globalShortcut, meaning the shortcut will be registered system-wide. In this case Cmd+Q is registered system-wide as quit for muffon and even when the user uses the keybinding on another app they want to close, the keybinding fires the quit function. Solution would be to NOT register the shortcut globally but locally instead.

@roloit
Copy link
Author

roloit commented Oct 19, 2024

I definitely was outside of muffon.
Thanks for finding the bug! I will probably use previous release right now.

@xyloflake
Copy link
Contributor

I definitely was outside of muffon. Thanks for finding the bug! I will probably use previous release right now.

Note: You don't have to wait for official releases. Our beta program offers facilities for testing the fixes (e.g., v2.1.0 had 23 beta tests). What you can do:

  1. Wait for our fix.
  2. Join the beta program.
  3. Use the beta version until stability is confirmed.

TL;DR Wait for us to fix this issue, enroll in the beta program and use the beta until we declare the fix stable. I do not see an official release coming in the next 8 weeks. But betas are quite frequent.

@xyloflake xyloflake added the bug label Oct 19, 2024
@xyloflake xyloflake changed the title muffon quits when I press force quit with shortcut (outside of the app) on macOS [bug] keybindings override system shortcuts Oct 19, 2024
@xyloflake xyloflake pinned this issue Oct 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants