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

alt + tab resets fullscreen mode #26

Closed
D1mon opened this issue May 22, 2018 · 5 comments
Closed

alt + tab resets fullscreen mode #26

D1mon opened this issue May 22, 2018 · 5 comments
Labels

Comments

@D1mon
Copy link

D1mon commented May 22, 2018

alt + tab resets fullscreen mode and does not get back up.
Reproduce the problem:
I opened the video player in full-screen mode and pressed alt + tab.

@drgibbon
Copy link

Awesome 4.2, can confirm that behaviour here also.

@blueyed
Copy link
Owner

blueyed commented Dec 27, 2018

This is because setting ontop = true for the currently selected client. See awesomeWM/awesome#667.

Please try #27.

An unsolved issue there still is that with a fullscreen and ontop client on a tiled layout, the fullscreen client will not be visible when selected.
A solution might be to set "ontop" then - ensuring basically that it becomes visible, but I am not sure if that's good?!

@blueyed
Copy link
Owner

blueyed commented Dec 27, 2018

Came up with awesomeWM/awesome#2512 for awesome itself - would be nice to have this in the next release, but a fix for this (#27) is still valid, of course.

The problem that a selected client might not be visible in the end is another issue though, and not really the responsibility of awesome-cyclefocus I guess. Still interested about ideas, of course.

blueyed added a commit that referenced this issue Dec 27, 2018
@blueyed
Copy link
Owner

blueyed commented Dec 27, 2018

JFI: #28 added support for a raise_client option that would allow to do something like checking if the client is visible (not trivial though I guess), and then also setting ontop.

@drgibbon
Copy link

After a pull to 127a082 the problem seems to be fixed, thanks!

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

No branches or pull requests

3 participants