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

fix(mux): fix edge pane detection logic for tmux (#118) #130

Merged
merged 1 commit into from
Oct 10, 2023
Merged

fix(mux): fix edge pane detection logic for tmux (#118) #130

merged 1 commit into from
Oct 10, 2023

Conversation

freddiehaddad
Copy link
Contributor

@freddiehaddad freddiehaddad commented Oct 9, 2023

Fixes #118

@freddiehaddad freddiehaddad changed the title fix(tmux): implement proper edge pane detection (#118) fix(mux): implement proper edge pane detection (#118) Oct 9, 2023
@mrjones2014 mrjones2014 changed the title fix(mux): implement proper edge pane detection (#118) fix(mux): fix edge pane detection logic for tmux (#118) Oct 10, 2023
lua/smart-splits/mux/tmux.lua Outdated Show resolved Hide resolved
@mrjones2014
Copy link
Owner

Wow, this seems much simpler! Thanks for the PR! I'll wait for @icyrainz to confirm this fixes the issue for him as well before merging.

@mrjones2014
Copy link
Owner

We're also getting lua/smart-splits/mux/tmux.lua:3:7: unused variable AtEdgeBehavior in CI now. Since its unused now, you can just delete the line.

Rather than using the format flag (-F) to find all panes and
attempting to detect whether or not the active pane is at an
edge, the filter flag (-f) can be used instead to conditionally
query a single result indicating if the active pane is at an
edge.
@mrjones2014 mrjones2014 merged commit dbf2c13 into mrjones2014:master Oct 10, 2023
2 checks passed
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

Successfully merging this pull request may close these issues.

[Bug]: Navigation via tmux not working after tmux pane resized
2 participants