Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Unintuitive navigation controls #1220

Closed
yuhan0 opened this issue Apr 3, 2022 Discussed in #1210 · 2 comments
Closed

Unintuitive navigation controls #1220

yuhan0 opened this issue Apr 3, 2022 Discussed in #1210 · 2 comments
Labels
bug Something isn't working

Comments

@yuhan0
Copy link

yuhan0 commented Apr 3, 2022

Discussed in #1210

I first posted about this in Discussions along with some ideas for improving the system, but I've since become convinced it's an actual issue that needs fixing.

Here's a short screen recording illustrating the problem in an easily reproducible situation.
(The only shortcuts used are window --focus west and window --focus east represented by the ← → arrows in the windows.)

yabai-nav2.mp4

Stage 1:
Focusing → then ← brings me to a different window than I started on. Not a big deal since the situation is 'ambiguous' due to the two possible targets. If Yabai resolves this consistently (it does not) I could get used to this.

 ┌──┬──┐
 │  │← │
 ├──┤  │
 │→ │  │
 └──┴──┘

Stage 2:
Everything works as expected

 ┌──┬──┐
 │  │  │
 ├──┼──┤
 │→ │← │
 └──┴──┘

Stage 3:
A small change from stage 2, yet now ← brings me to the window ? for seemingly no reason. This is the biggest issue which I run into on a daily basis, and it makes directional focusing/manipulation of windows hard to understand.

┌─┬─┬──┐
│ │?│  │
├─┴─┼──┤
│→  │← │
└───┴──┘

The linked discussion goes into a little more detail on possible solutions. I'd be willing to help contribute a fix, but unsure if others are facing the same issues.

@FelixKratz
Copy link

This bothered me as well, but can be fixed very easily: https://github.com/FelixKratz/yabai/commit/15adc077db5ccc866e5a6a887493e90d366c71df

@koekeishiya
Copy link
Owner

Implemented better detection logic on master.

@koekeishiya koekeishiya added the addressed on master; not released Fixed upstream, but not yet released label Apr 3, 2022
@koekeishiya koekeishiya added the bug Something isn't working label Apr 3, 2022
koekeishiya added a commit that referenced this issue Apr 28, 2022
@koekeishiya koekeishiya removed the addressed on master; not released Fixed upstream, but not yet released label May 17, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants