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

C-z (suspend-frame) crashes EXWM (solution provided) #36

Open
amirny2205 opened this issue May 17, 2024 · 5 comments
Open

C-z (suspend-frame) crashes EXWM (solution provided) #36

amirny2205 opened this issue May 17, 2024 · 5 comments
Labels
enhancement New feature or request

Comments

@amirny2205
Copy link

amirny2205 commented May 17, 2024

accidently pressing C-z or C-x C-z (bound to suspend-frame) crashes EXWM

solution:
(global-unset-key (kbd "C-z"))
(global-unset-key (kbd "C-x C-z"))

what would be the status for this thread "resolved but still accepts better answers"? probable org-mode...

@Stebalien
Copy link
Contributor

Hm. We should probably advise that function and fail if it's an Emacs frame.

@Stebalien Stebalien added the enhancement New feature or request label May 21, 2024
@minad
Copy link
Member

minad commented May 23, 2024

I usually suggest to unbind/rebind C-z anyway (for GUI Emacs), since suspending is only useful when Emacs is used on the terminal. So maybe it is sufficient to document this problem and recommend a useful alternative command for C-z?

@Stebalien
Copy link
Contributor

For suspend, yes. Although suspend-frame is still useful in GUI mode, just not for EXWM frames (it'll iconify GUI frames).

@minad
Copy link
Member

minad commented May 23, 2024

Yes, we never iconify, so the recommendation in our case would be even
stronger. :) But probably there is nothing wrong with installing :override
advices for commands or functions which must never be used in EXWM. The
advice could simply print a message or throw an error. We could
introduce a variable similar to exwm-input--passthrough-functions.

@Stebalien
Copy link
Contributor

Yes, we never iconify, so the recommendation in our case would be even stronger.

Yep. My point is that it's not a blanket "don't do this in a GUI window".

But probably there is nothing wrong with installing :override advices for commands or functions which must never be used in EXWM.

Or just have an :arround advice that checks if the frame is managed by EXWM (in theory, it would be nice to eventually support multiple X displays..., although I guess that kind of setup is basically dead at this point).

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

No branches or pull requests

3 participants