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

Toggle fullscreen feature #394

Closed
R4stafa opened this issue Oct 27, 2020 · 3 comments
Closed

Toggle fullscreen feature #394

R4stafa opened this issue Oct 27, 2020 · 3 comments

Comments

@R4stafa
Copy link

R4stafa commented Oct 27, 2020

I use XMonad.Layout.Fullscreen for fullscreen support and the AddFullscreen and RemoveFullscreen messages to manually make windows fullscreen but there is no easy way (at least I can't see one) to toggle the fullscreen state of a window say with a keybinding. So I went ahead and built my own xmonad-contrib where I added a ToggleFullscreen message and it works. I thought this feature should be in the official xmonad-contrib but I wanted to discuss this first before making a PR.

Sorry if I do something the the wrong way I am new to contributing to open source projects.

@elkowar
Copy link
Contributor

elkowar commented Oct 27, 2020

the whole point of contrib is that it's easy to PR to! So if you already have the code, there is no harm in simply doing the PR, and seeing what happens!

@R4stafa
Copy link
Author

R4stafa commented Oct 27, 2020

Ok I created the PR #395

@slotThe
Copy link
Member

slotThe commented Mar 6, 2022

At this point this is basically a duplicate of #456. This will be added once #625 is merged

@slotThe slotThe closed this as completed Mar 6, 2022
liskin added a commit to liskin/xmonad-contrib that referenced this issue Oct 23, 2022
Fairly straightforward, just add two hooks for (un)fullscreening. There
are multiple motivations for this:

* Users are calling for unfullscreened windows to revert back to their
  original location if they were floating.

* XMonad.Layout.Fullscreen uses some deprecated exports from
  XMonad.Hooks.EwmhDesktops and reimplements fullscreenEventHook.

This commit only adds the hooks. Neither of the motivations are dealt
with yet, so the docs are a bit terse still.

Related: xmonad#456
Related: xmonad#394
Related: xmonad#626
liskin added a commit to liskin/xmonad-contrib that referenced this issue Mar 25, 2023
Fairly straightforward, just add two hooks for (un)fullscreening. There
are multiple motivations for this:

* Users are calling for unfullscreened windows to revert back to their
  original location if they were floating.

* XMonad.Layout.Fullscreen uses some deprecated exports from
  XMonad.Hooks.EwmhDesktops and reimplements fullscreenEventHook.

This commit only adds the hooks. Neither of the motivations are dealt
with yet, so the docs are a bit terse still.

Related: xmonad#456
Related: xmonad#394
Related: xmonad#626
liskin added a commit to liskin/xmonad-contrib that referenced this issue Mar 25, 2023
liskin added a commit to liskin/xmonad-contrib that referenced this issue Mar 26, 2023
Fairly straightforward, just add two hooks for (un)fullscreening. There
are multiple motivations for this:

* Users are calling for unfullscreened windows to revert back to their
  original location if they were floating.

* XMonad.Layout.Fullscreen uses some deprecated exports from
  XMonad.Hooks.EwmhDesktops and reimplements fullscreenEventHook.

This commit only adds the hooks. Neither of the motivations are dealt
with yet, so the docs are a bit terse still.

Related: xmonad#456
Related: xmonad#394
Related: xmonad#626
liskin added a commit to liskin/xmonad-contrib that referenced this issue Mar 26, 2023
liskin added a commit to liskin/xmonad-contrib that referenced this issue Apr 1, 2023
Fairly straightforward, just add two hooks for (un)fullscreening. There
are multiple motivations for this:

* Users are calling for unfullscreened windows to revert back to their
  original location if they were floating.

* XMonad.Layout.Fullscreen uses some deprecated exports from
  XMonad.Hooks.EwmhDesktops and reimplements fullscreenEventHook.

This commit only adds the hooks. Neither of the motivations are dealt
with yet, so the docs are a bit terse still.

Related: xmonad#456
Related: xmonad#394
Related: xmonad#626
liskin added a commit to liskin/xmonad-contrib that referenced this issue Apr 1, 2023
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

No branches or pull requests

3 participants