Nice, thanks a lot! After a quick test I think it works again with Hyprland.

I just added a small comment on the code organization, let me know what you think.

@LorenzoBettini do you fancy a test as well? :)

@lbonn yes, I'd like to test it :)
what would be the best way to do that?

@LorenzoBettini

• git clone https://github.com/alebastr/rofi -b wayland-window-unmap
• git submodule update --init

Then follow the instructions in https://github.com/lbonn/rofi/blob/wayland/INSTALL.md (only meson will work, not autotools)

eg meson setup build, ninja -C build ...

@lbonn

here's what I've done:

sudo pacman -S meson

cd ~/tmp
git clone https://github.com/alebastr/rofi -b wayland-window-unmap
cd rofi
git submodule update --init

meson setup build
ninja -C build

build/rofi -version
Version: 1.7.5 wayland1-252-gc7472ecc (wayland-window-unmap)

Then I tried (with a few windows in separate hyprland workspaces):

build/rofi -show window

and selecting the window now correctly switches to it (in case, also switching to the corresponding workspaces).

Well done! :)
And thanks for fixing it so fast!

Great! Thanks for your help.

@alebastr is it fine by you if we merge this now?

Thanks for testing!

@lbonn from a brief look, the only public method that touches the surface and does not check it is display_set_surface_dimensions. I guess if that becomes a problem, we could fix it later.

Feel free to merge.

Thanks!

Just want to show my appreciation that running paru -S rofi-lbonn-wayland-git works!

Thank you for fixing it! I really appreciate it!

Thanks again!
Are you going to release this version?

@LorenzoBettini true, maybe not a bad idea. There have been quite a few fixes since last version.