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

[Mir-on-Wayland] On GNOME/Wayland all windows open on first monitor #1903

Closed
AlanGriffiths opened this issue Jan 28, 2021 · 5 comments · Fixed by #2562
Closed

[Mir-on-Wayland] On GNOME/Wayland all windows open on first monitor #1903

AlanGriffiths opened this issue Jan 28, 2021 · 5 comments · Fixed by #2562

Comments

@AlanGriffiths
Copy link
Contributor

With two monitors in use on GNOME/Wayland desktop run:

miral-app --wayland-host $WAYLAND_DISPLAY

Expect: a fullscreen miral-shell window appears on each output.
Actual: a fullscreen miral-shell window for each output appears the primary output.

@AlanGriffiths
Copy link
Contributor Author

AlanGriffiths commented Dec 14, 2021

A quick look at the logs shows the fullscreen requests have different wl_outputs:

$ WAYLAND_DEBUG=client egmde --wayland-host $WAYLAND_DISPLAY 2>&1 | grep -e wl_output@13 -e wl_output@11
[2672554.762] wl_output@11.geometry(1920, 0, 520, 290, 0, "ACI", "ASUS VS247", 0)
[2672554.779] wl_output@11.mode(3, 1920, 1080, 60000)
[2672554.786] wl_output@11.scale(1)
[2672554.788] wl_output@11.done()
[2672554.791] wl_output@13.geometry(0, 0, 520, 290, 0, "ACI", "ASUS VS247", 0)
[2672554.801] wl_output@13.mode(3, 1920, 1080, 60000)
[2672554.808] wl_output@13.scale(1)
[2672554.810] wl_output@13.done()
[2672562.166]  -> wl_shell_surface@19.set_fullscreen(1, 0, wl_output@11)
[2672562.218]  -> wl_shell_surface@17.set_fullscreen(1, 0, wl_output@13)

@AlanGriffiths
Copy link
Contributor Author

Note that it is possible to move a window between outputs (e.g. Meta+Shift+Right)

@RAOF
Copy link
Contributor

RAOF commented Dec 15, 2021

It would not surprise me at all if GNOME Shell simply ignores the output parameter. It would likewise not surprise me if Shell ignored the method parameter, but it might be worth checking if this works setting the method to driver.

@AlanGriffiths
Copy link
Contributor Author

It would not surprise me at all if GNOME Shell simply ignores the output parameter. It would likewise not surprise me if Shell ignored the method parameter, but it might be worth checking if this works setting the method to driver.

Worth a try, but doesn't help.

@AlanGriffiths
Copy link
Contributor Author

Fixed by #2437

bors bot added a commit that referenced this issue Aug 30, 2022
2562: Release 2.9.0 r=AlanGriffiths a=AlanGriffiths

## ABI summary:
* miral ABI bumped to 5
* mircommon ABI unchanged at 9
* mircookie ABI unchanged at 2
* mircore ABI bumped to 2
* miroil ABI bumped to 2
* mirplatform ABI unchanged at 23
* mirserver ABI unchanged at 58
* mirwayland ABI unchanged at 3
* mirplatformgraphics ABI unchanged at 20
* mirinputplatform ABI unchanged at 8
## Enhancements:
* [Wayland] Implement zwp_idle_inhibit_manager_v1
* [Wayland] Implement zwlr_virtual_pointer_v1
* [Wayland] Implement zwp_text_input_manager_v1 (Electron works with OSK)
* [Wayland] Bump wl_seat to v8 and implement hi-res scrolling (Fixes: #2176, Fixes: #2499)
* [Wayland platform] improve failed to connect error
* [Wayland platform] Port to xdg-shell (Fixes #1903, Fixes: #2434)
* [MirAL] Allow `--add-wayland-extenions all`
* [MirAL] Allow servers to get repeated string options
* [MirAL] Improvement to ExternalClientLauncher: Don't force clients to split command lines themselves
* [MirAL] Tidy up event filtering API
* [MirAL] Expose miral::Zone::id()
* [Input] Filter 2 distinct bogus touch event scenarios (UBports)
* [gbm-kms] new driver quirk to disable KMS modeset probe
* [gbm-kms] Add defaults to driver-quirks for nvidia and evdi (Fixes: #2467)
* [gbm-kms] Add defaults to driver-quirks for vc4-drm and v3d
* [mir-smoke-test-runner] Enable working in a Wayland only environment
## Bugs fixed:
* Fix ABI breakage of libmircore.so.1 with v2.8.0 vs v1.8.2 (Fixes #2465)
* [Wayland] Text Input v2: do not use commit count as serial
* [Wayland] Send keyboard modifiers after keyboard enter (Fixes: #2535, Fixes: #2025)
* [Wayland] wlr-screencopy-v1: send .damage event as required (wayvnc fix)
* [Wayland] wlr-screencopy-v1: Waits until the copy area has been damaged in `.copy_with_damage` request (wayvnc fix)
* [MirAL] Unblock signals before execing child processes (Fixes: #1284)
* [eglstream-kms] Kill clients, not Mir, when they submit bad EGLStreams (Fixes: #2061)
* [eglstream-kms] Handle EGL errors in devnum_for_device (Fixes: #2426)
* [test clients] Explicitly ask for a GLESv2 context (Fixes: #2440)
* [renderers/gl] Clear framebuffer to opaque black (Fixes: #2427)
* CMake cleanup for locally built dependencies (Fixes: #2507, Fixes: #2261)
* Fix event timestamps (Fixes: #2577)

Co-authored-by: Alan Griffiths <alan@octopull.co.uk>
Co-authored-by: bors[bot] <26634292+bors[bot]@users.noreply.github.com>
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

Successfully merging a pull request may close this issue.

2 participants