domferr / tilingshell

Extend Gnome Shell with advanced tiling window management. Supports multiple monitors, Windows 11 Snap Assistant, Fancy Zones, customised tiling layouts and more.
https://extensions.gnome.org/extension/7065/tiling-shell/
GNU General Public License v2.0
645 stars 20 forks source link

[BUG] Keybinding move left/right moves diagonally on even 2x2 tiling layout #157

Closed thyttan closed 3 weeks ago

thyttan commented 3 weeks ago

Describe the bug On an even 2x2 tiling layout using keybindings to move window horizontally moves them diagonally instead.

To Reproduce Steps to reproduce the behavior:

  1. On an empty workspace select an even 2x2 tiling layout.
  2. Launch a terminal window.
  3. Use keybinding Move window to right tile (I use alt + super + L).
  4. Window snaps to the top left. (Up for discussion if this is expected or unexpected behaviour)
  5. Use keybinding Move window to right tile again.
  6. Window snaps to top right.
  7. Use keybinding Move window to the left tile.
  8. Window snaps to bottom left. (Unexpected!)
  9. Use keybinding Move window to right tile again.
  10. Window snaps to top right. (Unexpected!)

The repro above is what I did in the screencast below.

Screencast --removed--

Information (please complete the following):

Additional context image

domferr commented 3 weeks ago

Hey thanks a million for reporting this! I'm still improving the algorithm behind all of this, so it's great to have your feedback. I made some changes, would you like to give it a try?

thyttan commented 3 weeks ago

Yes - that did the trick! Thanks!

domferr commented 3 weeks ago

Great! :star:

thyttan commented 3 weeks ago

While we're at it - I'll open another issue for a similar, but less clear cut case which I feel is unexpected but maybe is expected.

domferr commented 3 weeks ago

Ok thank you! Can you also check whether the issue occurs in the version I shared you above?

thyttan commented 3 weeks ago

Yes it does :) But again - it's not as clear cut unexpected as this issue was.