~tsdh/swayr#42: No switch to floating window

Hi, Since swayr 0.27.3, it seems that the following no longer switches against floating windows. Is this intentional?

Steps:

1. exec `foot --app-id=test`.
2. Make the above into a floating window
3. exec `swayr switch-to-app-or-urgent-or-lru-window --skip-urgent --skip-origin --skip-lru "test"`

-> Unfocused

Screencast: https://0x0.st/XmwE.mp4

Environment:

- swayr-git r492.b76e90e-1
- sway-git 1.10.r7352.fce8de0-1
- wlroots-git 0.18.0.r7093.6d197eef9-1
- OS: Arch Linux x86_64
- Kernel: 6.9.6-arch1-1

Status: RESOLVED CLOSED
Submitter: ~tkna
Assigned to: No-one
Submitted: 8 months ago
Updated: 8 months ago
Labels: not_a_bug

~tsdh added not_a_bug 8 months ago

~tsdh REPORTED CLOSED 8 months ago

Hi there, you have discovered a bug but not in swayr but foot. Check swayr get-windows-as-json or swaymsg -t -r get_tree and you will see that the foot window still has "app_id": "foot" and not "test". So it seems that foot's --app-id option has stopped working.

~tsdh CLOSED REPORTED 8 months ago

~tsdh 8 months ago

Oh, sorry, I got confused by another foot window and the fact that swayr get-windows-as-json also seems to ignore floating windows (which isn't expected, too).

~tsdh 8 months ago

Ok, the problem is that the new version 1.3.1 of the swayipc-types crate doesn't return floating nodes in Node::iter(). I've created a bug report there (https://github.com/JayceFayne/swayipc-rs/issues/53) to check if that's intended or a bug.

~tsdh 8 months ago

The bug on the other side has been fixed, so I only have to wait for a new release of the swayipc-types crate.

~tkna 8 months ago

It was good. Thank you!

~tsdh REPORTED CLOSED 8 months ago

I've released swayr-0.27.4 which uses the new version of swayipc-types where the bug is fixed.

~tkna 8 months ago

It seems to work fine. Thanks!