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

Kitty is not always tiled #179

Closed
ghost opened this issue Jul 31, 2019 · 11 comments
Closed

Kitty is not always tiled #179

ghost opened this issue Jul 31, 2019 · 11 comments
Labels
bug Something isn't working

Comments

@ghost
Copy link

ghost commented Jul 31, 2019

As you can see, Kitty is not always being tiled.

screenshot

I have not been able to observe a pattern. If closing all the Kitty windows and then opening more, proper tiling resumes.

@goranmoomin
Copy link

@danteissaias Have you run yabai -m query --windows --space to analyze the problem?

@ghost
Copy link
Author

ghost commented Jul 31, 2019

This is the value of one of the non-tiled windows:

{
        "id":31133,
        "pid":50498,
        "app":"kitty",
        "title":"htop /Users/dante",
        "frame":{
                "x":692.0000,
                "y":287.0000,
                "w":100.0000,
                "h":100.0000
        },
        "level":0,
        "role":"AXWindow",
        "subrole":"AXStandardWindow",
        "movable":1,
        "resizable":1,
        "display":1,
        "space":1,
        "visible":1,
        "focused":0,
        "split":"none",
        "floating":1,
        "sticky":0,
        "border":1,
        "shadow":0,
        "zoom-parent":0,
        "zoom-fullscreen":0,
        "native-fullscreen":0
}

For some reason it seems to be floating, but I don't know why.

@goranmoomin
Copy link

@danteissaias I'm not sure why that happens (maybe a config messed it up? or it's a bug?) but you can use the signal system to manually make the floating values to 0 :-)
I haven't touched the signal system for a while, but there are events that are triggered when new windows are created.

@ghost
Copy link
Author

ghost commented Jul 31, 2019

@pcr910303 This issue occurs with the example configuration, could this issue possible be a relative of #174, since maybe the Kitty windows are not always defloated?

@ghost ghost closed this as completed Jul 31, 2019
@ghost ghost reopened this Jul 31, 2019
@ghost
Copy link
Author

ghost commented Jul 31, 2019

If its any help I discovered the trigger of this issue. If you open 4 Kitty windows tiled as so:

screenshot

If you then create another Kitty window, and close it using CMD+W, the next Kitty window that is opened will suffer from this issue.

screenshot 2

Until closing the suffering window, and the last window created before that:

screenshot 3

All Kitty window subsequently created will not be tiled.

If you repeat these same steps, you can reproduce this issue on any application. I have tested this issue on Firefox Developer Edition and I can successfully reproduce it.

@koekeishiya
Copy link
Owner

Unable to reproduce using your steps. This may be a duplicate of #183 (?) which has been fixed on master.

In any case, can you reproduce this issue on the latest stable version (v1.1.2)?

@ghost
Copy link
Author

ghost commented Aug 1, 2019

It's not a duplicate of #183 as I am still able to produce this on master.

@ghost
Copy link
Author

ghost commented Aug 1, 2019

Here is a more detailed instruction on how to reproduce this issue, in case there was any confusion.

Edit: Here is a recording including keystrokes.

@koekeishiya
Copy link
Owner

I still can't reproduce this issue doing the exact steps shown in that video.

@koekeishiya koekeishiya added the undetermined Not sure if this is considered a real bug label Sep 3, 2019
@koekeishiya koekeishiya reopened this Feb 11, 2020
@koekeishiya
Copy link
Owner

So this is actually happening because there is a check for minimum window size upon creation, and if the window is smaller than the given dimensions, it will float instead: https://github.com/koekeishiya/yabai/blob/master/src/window.c#L263

I'm a bit reluctant to calling this a bug, but this is a sideeffect I had not considered..

@koekeishiya
Copy link
Owner

I think this should be considered a bug. We should only float small windows that are NOT resizable.

@koekeishiya koekeishiya added bug Something isn't working and removed undetermined Not sure if this is considered a real bug labels Feb 11, 2020
@koekeishiya koekeishiya added addressed on master; not released Fixed upstream, but not yet released and removed addressed on master; not released Fixed upstream, but not yet released labels Feb 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants