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

donotdisturb plugin: "Switch the interruptible windows to normal mode" does not work #653

Open
deltragon opened this issue Oct 2, 2024 · 0 comments · May be fixed by #652
Open

donotdisturb plugin: "Switch the interruptible windows to normal mode" does not work #653

deltragon opened this issue Oct 2, 2024 · 0 comments · May be fixed by #652
Assignees
Labels

Comments

@deltragon
Copy link
Collaborator

Describe the bug
The "Switch the interruptible windows to normal mode" setting for the donotdisturb plugin is supposed to un-fullscreen a fullscreen window if it is bypassed by the "Interrupt these windows regardless of their state" setting.

This does not work currently. The bypass works, and the break triggers despite the window being fullscreen, but it does not get un-fullscreened.

This only applies to X11 as Wayland currently does not have support for the "Interrupt these windows regardless of their state" and "Switch the interruptible windows to normal mode" settings.

To Reproduce
Steps to reproduce the behavior:

  1. Test on X11 (eg. LXQt).
  2. Enable the "Do not disturb" plugin.
  3. Fill the "Interrupt these windows regardless of their state" setting with an application (eg. qterminal).
  4. Ensure the "Switch the interruptible windows to normal mode" setting is set.
  5. Fullscreen the application selected in step 3 (eg. qterminal), and keep it focused.
  6. Trigger a break (this can be done from within qterminal, ensuring it stays focused).
  7. Notice how the break triggers, however the qterminal application stays fullscreen (possibly even covering the break screen).

Expected behavior
The application (qterminal) should be un-fullscreened.

Desktop (please complete the following information):

  • OS: Any (tested on Debian 12)
  • Desktop Env: X11 (eg. LXQt)
  • Version: Latest
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 a pull request may close this issue.

1 participant