make warp (banish-cursorwarp) work with the togglescratch function (renamedscratchpads)

[adetabrani]

simplescreenrecorder.webm

hi bak, how to make warp work with the togglescratch function? I tried adding the 'warp(c)' function call inside the togglescratch function, but it always ended up crashing 😂

[bakkeby]

Presumably you only want to call warp when the client is revealed?

[adetabrani]

yes, because of the current behavior I have to always refocus when the scratchpad window is toggled.

I want when I toggle scratchpad, the cursor automatically move to the scratchpad window, but I'm not sure what if there is more than 1 scratchpad window being managed, where should the cursor be located?

[adetabrani]

is there any feedback regarding this issue?

[bakkeby]

Have you given any more thought into your last question?

I'm not sure what if there is more than 1 scratchpad window being managed, where should the cursor be located?

If it were up to you and there were two scratchpad windows toggled together - where would you want the mouse cursor to warp to (if at all)?

[adetabrani]

thanks for the feedback, warp to the last focused scratchpad window (before hiding), to be honest I often use just one window as a scratchpad

[bakkeby]

Have you considered how the warp patch works for non-scratchpad clients? Does the mouse cursor stay with e.g. the last window spawned?

[adetabrani]

ah I almost forgot about that, maybe instead we can warp to the first stack on the scratchpad window (if more than 1 scratchpad window)? honestly I rarely set more than 1 scratchpad window, so I think at least warp should be able to work first for one scratchpad window