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

Disable splitting of floating frames #76

Merged
merged 1 commit into from
Aug 23, 2024

Conversation

Stebalien
Copy link
Contributor

Previously, floating frames were configured to never automatically split, but they could be manually split (usually by accident).

  • exwm-floating.el (exwm-floating--set-floating): disable splitting of floating windows by setting a the `split-window' window parameter to a function that always throws an error.

Previously, floating frames were configured to never automatically
split, but they could be manually split (usually by accident).

* exwm-floating.el (exwm-floating--set-floating): disable splitting of
  floating windows by setting a the `split-window' window parameter to a
  function that always throws an error.
@minad
Copy link
Member

minad commented Aug 23, 2024

I am not sure. Maybe one can consider this a feature, not a bug? Otoh floating windows are mostly temporary dialogs, so allowing splits may lead to confusion.

(#63 seems related)

@Stebalien
Copy link
Contributor Author

At the moment we're in kind of an awkward world where floating frames are mostly dedicated, but not 100%. I'd prefer to just make it consistent.

In terms of #63, I'd make those frames unsplittable as well (just with an Emacs buffer instead of an X buffer).

@minad
Copy link
Member

minad commented Aug 23, 2024

Okay, sounds good. Let's go with your patch then.

@Stebalien Stebalien merged commit f453664 into master Aug 23, 2024
@Stebalien Stebalien deleted the steb/reject-floating-window-split branch August 23, 2024 19:08
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 this pull request may close these issues.

2 participants