Layout coding: hiding a window about penrose HOT 6 CLOSED

I think the window sizes have to be strictly > 0 (though I might be wrong: the XCB docs are patchy to say the least!)

I do know that for the window manager check window (for setting ewmh props etc) I set the width and height to 1px each and that effectively hides the window. Might be worth trying that?

If not, then I suspect I'd need to modify the layout API / ResizeAction to allow for unmapping a window as opposed to positioning it. The 1px trick feels pretty hacky to be honest so it might be worth doing that anyway...something like this with None indicating that the window should be unmapped. (Changing to another layout should remap it back but I suppose that'd then be something we'd need to check for / track).

pub type ResizeAction = (WinId, Option<Region>)

from penrose.

I tried changing the size to (1, 1) and I noticed a thing: it was already working before!

The problem actually is that the layout function doesn't seem to run when simply cycling though windows.

Nonetheless, I think the unmapping of windows might be better in the future. Apparently qutebrowser crashes when its screen size is as small as 0 or 1 (still investigating) and other applications might end up having the same bug.

from penrose.

@YohananDiamond are you able to check if this is what you are after?

from penrose.

The problem seems to be still here - I can focus other windows other than the first one, but they are on the background. But I noticed one thing: if I switch workspaces and go back, the window that was hidden is shown. Maybe the layout is not updating on focus change?

from penrose.

Ah sorry, yes you need to set the layout config to trigger follow focus like this. You probably also want to do what I've done and allow client wrapping as well (default is to allow but you need to make sure you specify it when you create the config)

Layout functions by themselve can't determine when and how they are triggered: they only specify what to do with the clients when they are triggered. To be honest, at least for the default layouts I probably should provide some helper functions that bundle them with the appropriate LayoutConfig.

from penrose.

Oh, okay. It's working now, thanks for the help!

from penrose.