Hyprland icon indicating copy to clipboard operation
Hyprland copied to clipboard

Add movewindowinv dispatcher

Open toasterofbread opened this issue 2 years ago • 13 comments

Adds the movewindowinv dispatcher, which is similar to movewindow but moves all windows within the active workspace except the active window (and takes only a monitor as its argument). Useful for when you want to quickly give space to particular window.

This also moves most of the code from the moveActiveToWorkspace method into a new method so that it can be used by the new dispatcher. I've tested both the movewindowinv dispatcher and the affected movewindow dispatcher.

Example config:

bind=SUPER,left,movewindowinv,l
bind=SUPER,right,movewindowinv,r

Demo video using the above config (I couldn't get the second monitor to be recorded, but that's where the windows are going):

https://user-images.githubusercontent.com/44363049/173507834-f6742ead-b32b-4622-801f-e3a143ea57e3.mp4

toasterofbread avatar Jun 12 '22 13:06 toasterofbread

waiting for checks to succeed, aka freedesktop gitlab to relive itself

vaxerski avatar Jun 13 '22 07:06 vaxerski

can someone like @fufexan or @ThatOneCalculator verify this? thanks.

vaxerski avatar Jun 13 '22 16:06 vaxerski

Testing now

ThatOneCalculator avatar Jun 13 '22 16:06 ThatOneCalculator

Doesn't work, has no effect.

ThatOneCalculator avatar Jun 13 '22 16:06 ThatOneCalculator

Doesn't work, has no effect.

Strange, are you sure you're using the inv dispatcher and not the original movewindow one? This still works fine for me when checked out to the movewindowinv branch.

toasterofbread avatar Jun 14 '22 05:06 toasterofbread

Added a brief demonstration video to the original post

toasterofbread avatar Jun 14 '22 06:06 toasterofbread

I made sure I was using the right option.

ThatOneCalculator avatar Jun 14 '22 07:06 ThatOneCalculator

I made sure I was using the right option.

Can you check the log to see if it's being triggered at all, or if there's an error?

toasterofbread avatar Jun 14 '22 08:06 toasterofbread

@fufexan maybe you can check this?

vaxerski avatar Jun 20 '22 13:06 vaxerski

Yep, works as expected.

fufexan avatar Jun 20 '22 14:06 fufexan

Any progress on this?

yavko avatar Aug 13 '22 08:08 yavko

Any progress?

nyabinary avatar Aug 23 '22 20:08 nyabinary

Any progress v2? If you wont continue developing please say so, I'm sure someone else will be willing to complete the rest of this PR.

yavko avatar Sep 21 '22 23:09 yavko

Should i fix this?

Dickby avatar Sep 25 '22 18:09 Dickby

Should i fix this?

It seems the developer has abandoned the PR so if you want to go ahead, no one is stopping you

yavko avatar Sep 25 '22 18:09 yavko

I will close this one. If anyone wants to make a similiar one, feel free.

vaxerski avatar Sep 25 '22 18:09 vaxerski