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

Revert Global Shortcut blocking on Wayland #5454

Closed
Max-FT opened this issue Jan 18, 2022 · 9 comments · Fixed by #5688
Closed

Revert Global Shortcut blocking on Wayland #5454

Max-FT opened this issue Jan 18, 2022 · 9 comments · Fixed by #5688
Labels
client feature-request This issue or PR deals with a new feature linux

Comments

@Max-FT
Copy link

Max-FT commented Jan 18, 2022

The issue

#5307 breaks shortcuts for Wayland users. Users opened numerous issues regarding global shortcuts not working outside the window on Wayland. However, there's a full workaround. Wayland users can add these two lines under the [shortcut] section in
Mumble.conf:

[shortcut]
linux\evdev\enable=true
linux\evdev\framesperpacket=1

I have personally used this workaround for the last year just fine on Wayland with both 1.3.4 and the git build of 1.4 before the team released it. Please consider reverting the commit blocking shortcut access and update your documentation to include this
workaround. As an aside, I'd like to thank the Mumble contributors for 1.4. It's looking like a solid release otherwise.

Mumble version

1.4.230

Mumble component

Client

OS

Linux

Additional information

No response

@Max-FT Max-FT added the support label Jan 18, 2022
@davidebeatrici
Copy link
Member

We should enable Evdev by default and eventually add an option in the settings.

@Krzmbrzl Krzmbrzl added the feature-request This issue or PR deals with a new feature label Jan 18, 2022
@Krzmbrzl
Copy link
Member

The commit should not simply be reverted as it fixed a specific issue that we should not re-introduce. In addition to that, for all I know in order to use evdev, the user has to be in some special group, which most users will not be in by default. Thus, for most users I think simply disabling the shortcuts on Wayland until we have implemented proper Wayland support is the best thing to do. Everything else will just lead to loads of bug reports of broken shortcuts.

That being said, I think we could improve upon the current situation by introducing an option to override this disablement for users that know how to make evdev work (we could even describe how to get it to work on our website and then link to that).

@Max-FT
Copy link
Author

Max-FT commented Jan 22, 2022

For any Wayland users who use push-to-talk and are struggling in the meantime, there's still a workaround. Follow the instructions at my handbook: https://maxft.gitlab.io/mumble-docs/chapter_4/settings/shortcuts.html#global-keyboard-shortcuts

Krzmbrzl added a commit to Krzmbrzl/mumble that referenced this issue May 18, 2022
In mumble-voip#5307 we disabled global shortcuts on Wayland, but we did it in a way
that would prevent anyone from re-enabling the support.

Since we have received reports of some users actually making use of the
shortcut system on Wayland, even without proper Wayland support, we
decided to remove the hard-lock and allow these users to re-enable the
shortcuts, if they so want.

Fixes mumble-voip#5454
@Krzmbrzl
Copy link
Member

@Max-FT could you please check whether #5688 addresses this issue in a suitable way?
The shortcuts should be disabled by default but you should be able to manually enable them again through the settings.

@Max-FT
Copy link
Author

Max-FT commented May 21, 2022

I fetched your PR and rebuilt. I don't see any option to re-enable the shortcuts menu.
2022-05-21_11-05-1653151710

Krzmbrzl added a commit to Krzmbrzl/mumble that referenced this issue May 23, 2022
In mumble-voip#5307 we disabled global shortcuts on Wayland, but we did it in a way
that would prevent anyone from re-enabling the support.

Since we have received reports of some users actually making use of the
shortcut system on Wayland, even without proper Wayland support, we
decided to remove the hard-lock and allow these users to re-enable the
shortcuts, if they so want.

Fixes mumble-voip#5454
@Krzmbrzl
Copy link
Member

Are you sure that you actually fetched the changes from my PR? 'cause with those changes these settings shouldn't be disabled anymore (I completely deleted the code part responsible for deactivating the settings box)... 🤔 👀
I did miss a second code part in the backend though which would prevent the shortcuts from working. I fixed that now. Could you just try again and see if that was what was preventing you from using shortcuts?

@Krzmbrzl
Copy link
Member

Ho - I just realized that we don't even seem to expose the setting that enabled/disables the shortcuts to the user. So with the default being off now, it seems there is indeed no way in the UI to turn them back on.
I'll have to add an option for that 👀

@Krzmbrzl
Copy link
Member

@Max-FT long time no see (sorry for that). I have updated #5688 and now it should be possible for you to switch the shortcut engine on and off to your heart's content. The default should be off, if you're on Wayland but enabling it should be as simple as clicking on the checkbox in the Shortcuts settings page.
Could you test this and tell me whether it works as intended?

@Max-FT
Copy link
Author

Max-FT commented Jul 30, 2022

Thank you very much. I fetched your PR and it works!

Krzmbrzl added a commit to Krzmbrzl/mumble that referenced this issue Jul 31, 2022
In mumble-voip#5307 we disabled global shortcuts on Wayland, but we did it in a way
that would prevent anyone from re-enabling the support.

Since we have received reports of some users actually making use of the
shortcut system on Wayland, even without proper Wayland support, we
decided to remove the hard-lock and allow these users to re-enable the
shortcuts, if they so want.

Fixes mumble-voip#5454
Krzmbrzl added a commit that referenced this issue Jul 31, 2022
In #5307 we disabled global shortcuts on Wayland, but we did it in a way
that would prevent anyone from re-enabling the support.

Since we have received reports of some users actually making use of the
shortcut system on Wayland, even without proper Wayland support, we
decided to remove the hard-lock and allow these users to re-enable the
shortcuts, if they so want.

Fixes #5454
Krzmbrzl added a commit to Krzmbrzl/mumble that referenced this issue Aug 10, 2022
In mumble-voip#5307 we disabled global shortcuts on Wayland, but we did it in a way
that would prevent anyone from re-enabling the support.

Since we have received reports of some users actually making use of the
shortcut system on Wayland, even without proper Wayland support, we
decided to remove the hard-lock and allow these users to re-enable the
shortcuts, if they so want.

Fixes mumble-voip#5454

(cherry picked from commit 49cc697)
Krzmbrzl added a commit to Krzmbrzl/mumble that referenced this issue Aug 10, 2022
In mumble-voip#5307 we disabled global shortcuts on Wayland, but we did it in a way
that would prevent anyone from re-enabling the support.

Since we have received reports of some users actually making use of the
shortcut system on Wayland, even without proper Wayland support, we
decided to remove the hard-lock and allow these users to re-enable the
shortcuts, if they so want.

Fixes mumble-voip#5454

(cherry picked from commit 49cc697)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
client feature-request This issue or PR deals with a new feature linux
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants