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

mpv-0.30.0 shows no window border in gnome-3.34, weston-7.0.0, wayland #7136

Closed
soloturn opened this issue Nov 9, 2019 · 13 comments
Closed

Comments

@soloturn
Copy link

soloturn commented Nov 9, 2019

with mpv-0.30.0 the window border is missing, with v0.29.1 it is working, just like #3350, on arch linux, gnome 3.34, wayland. on X it is always working.

@CounterPillow
Copy link
Contributor

CounterPillow commented Nov 9, 2019

Follow the actual issue template or we won't be able to help you.

Also looks like what you mean is that there is no border, which is expected behaviour on GNOME Wayland as it is a GNOME bug. Report it to them.

EDIT: Maybe to add to this, the reason why you are not seeing this in 0.29.1 is because 0.29.1 defaulted to the xwayland context. However, native wayland contexts in mpv now have significant enough advantages over xwayland (e.g. presentation feedback) that it would be unreasonable to make all wayland users have a suboptimal experience out of the box just because of the political games played by GNOME. Switching to a different Wayland compositor that actually respects its users and third-party developers is free.

@soloturn
Copy link
Author

soloturn commented Nov 9, 2019

mpv does not show a border in weston-7.0.0 just like it does not in gnome, i adjusted the title, @CounterPillow.

$ yay -Q | grep waylan
qt5-wayland 5.13.2-1
wayland 1.17.0-1
wayland-protocols 1.18-1
xorg-server-xwayland 1.20.5-4

@soloturn soloturn changed the title gnome 3.34, wayland no window mpv-0.30.0 shows no window border in gnome-3.34, weston-7.0.0, wayland Nov 9, 2019
@Dudemanguy
Copy link
Member

Dudemanguy commented Nov 10, 2019

Weston also doesn't support xdg-decoration protocol. In their case, it's just a reference compositor so it's a reasonable choice to make. Regardless, mpv is not in the business of drawing window decorations. It's up to the compositor to support this. For instance, Plasma and Sway both have window decorations.

@soloturn
Copy link
Author

@Dudemanguy thank you for the pointer, now i found the relevant piece in gnome mutter:
https://gitlab.gnome.org/GNOME/mutter/issues/217

@fmuellner was commenting that applications not supporting client side decorations are not following the wayland specification. citing out of https://github.com/wayland-project/wayland-protocols/blob/master/unstable/xdg-decoration/xdg-decoration-unstable-v1.xml

This interface allows a compositor to announce support for server-side
decorations.

[...]

If compositor and client do not negotiate the use of a server-side
decoration using this protocol, clients continue to self-decorate as they
see fit.

[...]

The decoration object allows the compositor to toggle server-side window
decorations for a toplevel surface. The client can request to switch to
another mode.

[...]

The compositor can decide not to use the client's mode and enforce a
different mode instead.

enlightment, gnome, weston did not implement server side decorations up to now. if it is not in the specification, it might be a little complicated to argue with gnome persons and tell them that it is a bug, @CounterPillow . but i understand your position, now than i read @bearoso, @jwrdegoede, @rcgordon in the resp gnome ticket.

@CounterPillow
Copy link
Contributor

if it is not in the specification, it might be a little complicated to argue with gnome persons and tell them that it is a bug,

Server side decorations are part of the spec.

GNOME just wants everyone to use GTK, there is no other reason for them not supporting it.

@soloturn
Copy link
Author

@Dudemanguy
Copy link
Member

There's actually CSDs now thanks to #7186.

@Rafostar
Copy link

@soloturn You can get decorations by forcing mpv to use XWayland via --gpu-context=x11egl option. This also works with hardware acceleration.

@soloturn
Copy link
Author

@Dudemanguy @philipl to do this via OSC, https://mpv.io/manual/master/#on-screen-controller, is extremely cool. works great.

@soredake
Copy link
Contributor

libsdl-org/SDL#4068

@soredake
Copy link
Contributor

Can mpv support this through https://gitlab.gnome.org/jadahl/libdecoration?

@Akemi
Copy link
Member

Akemi commented Feb 25, 2021

can, sure. want, probably not. we already have CSD now.

@Dudemanguy
Copy link
Member

Nobody really wants to incur a dbus dependency so almost certainly no.

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

No branches or pull requests

6 participants