Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

position_window_absolute might not work in all window managment schemes #273

Open
Tracked by #278
i509VCB opened this issue Feb 8, 2023 · 3 comments
Open
Tracked by #278

Comments

@i509VCB
Copy link

i509VCB commented Feb 8, 2023

WLCS at the moment uses position_window_absolute to place a window somewhere in the compositor coordinate space. However this type of behavior isn't going to be supported by every type of window management solution inside the compositor.

Tilers come to mind for an example of where placing a window at an absolute position in wlcs would conflict with how window management is done.

I guess the question then is how would position_window_absolute be adjusted to handle both types of window management concepts (tiling and floating) or would some tests like pointer_movement need a tiling mode?

@wmww
Copy link
Contributor

wmww commented Feb 8, 2023

Currently the assumption that WLCS can place a surface at an arbitrary location is baked pretty deep into a lot of the tests, and it would be hard to adapt some of them to not require this assumption. WLCS is focused more towards testing flexible display server libraries (like Mir and wlroots) and less towards specific shells (like Ubuntu Frame or Sway), so we hope this isn't too much of a problem.

@AlanGriffiths
Copy link
Contributor

I agree this is an issue, and would support a merging a solution if one were proposed.

In practice, this hasn't been enough of a problem to motivate anyone to find a solution.

@RAOF
Copy link
Contributor

RAOF commented Feb 9, 2023

Hm. Probably the way to deal with this is to invert control? Rather than having position_window_absolute and then the tests using hard-coded positions, have a layout_of_window query that returns the window rectangle, in compositor coordinates.

That still assumes windows map to rectangles, which is an assumption but probably a fairly reasonable one.

Avoiding races would be the trick with such a solution. I agree with Alan; this is a sensible thing to want, but not something that's an urgent priority for us.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants