djpohly/dwl

dwl does not allow xwayland clients to change override_redirect flag

Closed this issue · 10 comments

Info

dwl version: HEAD:wlroots-next (24a337e)
wlroots version: HEAD (05dd08af)

Description

It appears that Xwayland is no longer able to reason whether a pop-up window is an actual window or not, and will focus/unfocus the window as if it were an actual window. This seems to cheese REAPER's logic somehow (at least that's what it looks like to me). This is a video of what's happening (with expanded boarder size to illustrate the problem), first I tried to access the context menu with normal mouse settings, then I tried it with mouse-follows-focus turned on

2023-06-12.09-58-00.mp4

P.S, I was able to reproduce this on the main line (dwl HEAD:wlroots-next, wlroots HEAD) with all my mods and patches stripped

sevz17 commented

Where can I find the source code/binaries of the client?

For REAPER? You can find downloads on https://reaper.fm, or you can install the Arch package reaper

sevz17 commented

This is troublesome, seems like reaper creates popups with the override redirect flag turned off, and then turn it on.

I don't know it it's any help, but Sway doesn't seem to have any problems with the flag and it's also written in C...
Reaper seems to work flawlessly on Sway, especially the git version. Maybe there could be some clues in their source code to solving this...?

REAPER also seems to work in Hyprland

Yeah, with a couple of window rules added... If stock in it makes plugins and sends disappear if you move the mouse away from the plugin popup.

Here is the Sway issue about this. Looks like they removed something from their xwayland.c file.
swaywm/sway#6324
There's a whole bunch of info in there, don't know if any of it applicable to dwl but maybe some basic wayland stuff can hold some clues to solving this.

sevz17 commented

@StratusFearMe21 @Twinkie001, reaper now works on the wlroots-branch (https://codeberg.org/dwl/dwl/src/branch/wlroots-next)

Twinkie001 thank you very much for trying to give solutions. :)
StratusFearMe21 thanks for reporting.

@StratusFearMe21 @Twinkie001, reaper now works on the wlroots-branch (https://codeberg.org/dwl/dwl/src/branch/wlroots-next)

Twinkie001 thank you very much for trying to give solutions. :) StratusFearMe21 thanks for reporting.

Thank you! Can confirm that whatever you did worked!