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

Going full screen swaps side by side windows #188

Closed
tomfran opened this issue Mar 22, 2024 · 4 comments
Closed

Going full screen swaps side by side windows #188

tomfran opened this issue Mar 22, 2024 · 4 comments

Comments

@tomfran
Copy link

tomfran commented Mar 22, 2024

Hi, I noticed a problem on side by side windows after going full screen with one of them.

Basically, I started with the video on the left, went full screen, pressed esc, and the original placement swapped.


Initial state

Screenshot 2024-03-22 at 18 45 40

Full Screen on the video

Screenshot 2024-03-22 at 18 45 48

Press esc

Screenshot 2024-03-22 at 18 45 56

@nikitabobko
Copy link
Owner

nikitabobko commented Mar 22, 2024

It is by design (kinda)

When you fullscreen the window, the window detaches from the tree. When you exit fullscreen, the window attaches back to the tree

When the window attaches back to the tree, we need to find the position where to attach it. The current algorithm is to attach it after the most recently used window in the tree

In general case, after you detach window from the tree, the tree could be changed in an unpredictable way, so when you want to attach the window back, there is no such thing as "windows previous position", the tree has changed

In version 0.8.x, AeroSpace didn't detached and attached windows but kept fullscreen windows in the tree. Keeping the windows in the tree leads to phantom "holes" in the tree. See #18

@tomfran
Copy link
Author

tomfran commented Mar 22, 2024

Oh I see, thank you for the quick response!

@mpasa
Copy link

mpasa commented Oct 4, 2024

This happens for fullscreen video players and it's quite irritating. For example, if I have a browser with a YouTube page split like this.

+------------------------------------+---------------+
|              YouTube               |   Terminal    |
|               (70%)                |     (30%)     |
|                                    |               |
|                                    |               |
|                                    |               |
+------------------------------------+---------------+

Now, if you make the video on YouTube full screen and you close it, instead of going back to the previous layout, you will get:

+--------------------------+--------------------------+
|       Terminal           |          YouTube         |
|         (50%)            |          (50%)           |
|                          |                          |
|                          |                          |
|                          |                          |
+--------------------------+--------------------------+

@bekaman7
Copy link

I second this. It's not very nice when you transform your browser to be at a certain position + of a certain size and then lose all that by simply fullscreening a video.

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

4 participants