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

Golden ration stops working after :FocusEqualise #123

Open
eddyekofo94 opened this issue Aug 22, 2023 · 4 comments
Open

Golden ration stops working after :FocusEqualise #123

eddyekofo94 opened this issue Aug 22, 2023 · 4 comments

Comments

@eddyekofo94
Copy link

since I updated the today there seems to be alot of bugs. There was a refactoring lately which seems to have broken a few things and one of them is after a equalise the windows the golden ration seems to not work anymore.

@cryptomilk
Copy link
Collaborator

I'm sorry but I don't understand what exactly is the issue. Can you provide more detailed steps how to reproduce this? We need to write tests for more scenarios or we keep breaking them. Contributions are also welcome ;-)

@eddyekofo94
Copy link
Author

I was going to make a few PRs for some fixes but as you know... I got some code to deliver, maybe after I will look into it.

So, after I use the FocusMaxOrEqual the golden ration seems to not work anymore (the windows don't adjust to different sizes anymore)

hopefully that makes sense.

@willothy
Copy link
Contributor

willothy commented Aug 24, 2023

Ah, I see what you mean, the size check doesn't occur each time a new split is added or the window is resized. Will submit a PR for this soon.

Actually, I think the above is the intended behavior - MaxOrEqual either enables maximize or equalize modes based on the size of the current window. @cryptomilk mind confirming this?

If other behavior is wanted, it should likely be added as an additional feature.

However, it is true that autoresize breaks after max_or_equal. Working on a fix now.

For now, you can use FocusAutoresize to return to golden ratio mode.

Edit: Autoresize actually works fine for me after maximize, it was just an issue with my keymap lol. Not sure what the issue is here. I think you may just not be using FocusAutoresize or require("focus").focus_autoresize() to return to golden ratio mode.

@willothy
Copy link
Contributor

This should be fixed by #128, let me know if it wasn't.

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