-
-
Notifications
You must be signed in to change notification settings - Fork 31
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
Script not working properly on ultrawide monitors? #51
Comments
Hi @juipeltje, you are right; Line 106 in 31207c0
So with an ultra wide (21:9) the first two windows will be roughly 10.5:9, so still wider than high, thus resulting not in changing the split and affecting the next window. PR #44 aims to make it configurable which --splitratio .
|
ah i see, thanks for explaining it. so the |
Yeah, sorry for making you wait. I still have too many things to remember. I'll try to get back to it soon. |
no worries, i was just a bit confused about whether it was implemented or not, but it's nice to hear that it's coming at some point in the future. i messed around with some other autotiling scripts that are out there as well, but they all seem to suffer from the same issues when used on an ultrawide monitor. in the meantime i'll try to get used to manual tiling 😅 |
Personally I don't expect it to resolve the widescreen issue, but it's easy to test. Open the merged version of the script, save somewhere on $PATH as e.g. |
i just tried the merged script, but like you said, the behaviour is still the same. i'm not getting much output from the terminal other than
|
For more output use the |
ah yes, i see it now, it indeed does not say anything usefull, i just realized though when you mentioned the flag, that i was running the new script without any flags at all. i just tried it again with |
Give it a longer try, and then tell me your final opinion. |
alright, will do, i'll keep using it and see what happens. |
i just wanted to give a quick update after using the script for a few weeks now. so far i've only been using it with the
so far i haven't been able to tell what triggers this error, since it only happened twice, but i thought maybe it was worth mentioning. i'm not entirely sure if that's a problem with the script or if i'm missing some sort of python module, cause i installed i3ipc in a python venv and use the script that way. this error hasn't had any effect on how the script functions from what i can tell. the autotiling still worked as expected even after the error. |
If I understand well, you recommend to merge the changes, right? |
yes, i would say it's good to go. i haven't tested every single option it has because i wouldn't use them normally, but it hasn't given me any trouble and has been working perfectly so far. |
Came here looking for such a solution. The extra parameters look very useful, too. |
Pending PR merged. |
hello,
i'm using 2 ultrawide monitors and tried to use autotiling with sway, but the container never starts vertical splitting until a third column is opened, even when i run it with
--limit 2
. i'm assuming this has something to do with how the script determines when to split based on width/height, and the extra width of an ultrawide monitor probably messes that up. is there anything that can be done to solve that? i do use dynamic tilers to solve this, but i still love i3/sway as well, but the manual tiling is killing me lol.The text was updated successfully, but these errors were encountered: