<div dir="ltr">Hi,<div><br></div><div>I agree with your point regarding a SSD-capable compositor not always wanting them, and certainly I can see the usefulness for cases such as what you've cited. However, in the example you provided, it's easy enough for an application to determine what desktop it's running in and then determine whether to use SSD--if the compositor implements and advertises this protocol then it's ultimately a client decision whether to use it.</div><div><br></div><div>My issue is in the inconsistency of the protocol that has been proposed here. This is a protocol for enabling SSD; there should be no need for anything related to CSD, and I don't think there is a need for clients to have any form of protocol request to toggle between CSD and SSD.</div><div><br></div><div>Based on your assertion that borderless mode would be achieved in this protocol in the CSD mode, I would raise the counterpoint that in this case the client should just destroy the zxdg_toplevel_decoration_v1 object which, according to the destroy request, would revert to the surface using CSD anyway, furthering my assessment that there is indeed no need for any mention of CSD in the protocol requests.</div><div><br></div><div>This change allows for everything except the destroy request to be removed from <span style="font-size:13px;color:rgb(33,33,33)">zxdg_toplevel_decoration</span><span style="font-size:13px;color:rgb(33,33,33)">_v1, greatly simplifying the protocol as well as implementations of it. The case of the compositor "ignoring" the request to create SSD also becomes moot as well, since all that would mean is the compositor has chosen to use a borderless state at this time (e.g., tiling layout) which is irrelevant to the client anyway.</span></div><div><span style="font-size:13px;color:rgb(33,33,33)"><br></span></div><div><span style="font-size:13px;color:rgb(33,33,33)">This aside, there's some other items of note here, such as the lack of precision related to </span><span style="color:rgb(33,33,33)">when the change in decoration management state takes effect: most likely it should be applied on the next surface commit? I think this should be specified in the </span><span style="font-size:13px;color:rgb(33,33,33)">zxdg_toplevel_decoration</span><span style="font-size:13px;color:rgb(33,33,33)">_v1 interface description. Also, I would think that destroying the </span><font color="#212121">zxdg_toplevel_decoration_manager_v1 object while any </font><span style="font-size:13px;color:rgb(33,33,33)">zxdg_toplevel_decoration</span><span style="font-size:13px;color:rgb(33,33,33)">_v1 objects exist should be a client error, but this is similarly not specified anywhere.</span></div><div><span style="font-size:13px;color:rgb(33,33,33)"><br></span></div><div><span style="font-size:13px;color:rgb(33,33,33)">Regards,</span></div><div><span style="font-size:13px;color:rgb(33,33,33)">Mike</span></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Mar 1, 2018 at 1:18 PM Simon Ser <<a href="mailto:contact@emersion.fr">contact@emersion.fr</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Mike,<br>
<br>
I don't think a compositor supporting the protocol means that it always wants SSDs. For instance, I can imagine a DE like GNOME supporting it:<br>
- Allowing clients that prefer SSDs to use SSDs, so that toolkits like GLFW or winit and apps like mpv don't have to draw decorations that'll look alien and don't respect the user's preferences<br>
- But still preferring CSDs, so that GTK+ apps can use them<br>
<br>
Also, the CSD mode allows the client to do not draw decorations at all.<br>
<br>
Regards,<br>
<br>
On March 1, 2018 7:01 PM, Mike Blumenkrantz <<a href="mailto:michael.blumenkrantz@gmail.com" target="_blank">michael.blumenkrantz@gmail.com</a>> wrote:<br>
> Hi,<br>
><br>
> This patch is certainly an improvement upon the previous protocol draft which I reviewed, but it still does not address the most significant issue that I pointed out, which is that it both is too complex and lacks features. Why is there any "mode" when this is a protocol for enabling server-side decorations? By using the protocol in a server or client, you are enabling server-side decoration; why would there be any mention of client-side at all? If the compositor, for whatever reason, decides to do a runtime disable of SSD then it can simply destroy the global.<br>
><br>
> Please see again my mail on this topic from the previous thread: <a href="https://lists.freedesktop.org/archives/wayland-devel/2018-January/036495.html" rel="noreferrer" target="_blank">https://lists.freedesktop.org/archives/wayland-devel/2018-January/036495.html</a><br>
><br>
> Regards,<br>
><br>
> Mike<br>
><br>
> On Sun, Feb 18, 2018 at 3:01 PM Simon Ser <<a href="mailto:contact@emersion.fr" target="_blank">contact@emersion.fr</a>\> wrote:<br>
><br>
> > This adds a new protocol to negotiate server- and client-side rendering of<br>
> ><br>
> > window decorations for xdg-toplevels.<br>
> ><br>
> > \-\-\-<br>
> ><br>
> > This is inspired by a protocol from KDE\[0\] which has been implemented in KDE<br>
> ><br>
> > and Sway and was submitted for consideration in 2017\[1\]. This patch provides an<br>
> ><br>
> > updated protocol with those concerns taken into account.<br>
> ><br>
> > This was iterated on privately between representatives of Sway and wlroots<br>
> ><br>
> > (Simon Ser and Drew DeVault), KDE (David Edmundson), and Mir (Alan Griffiths).<br>
> ><br>
> > A proof-of-concept of a client and server implementation is available at \[2\].<br>
> ><br>
> > \[0\] <a href="https://github.com/KDE/kwayland/blob/master/src/client/protocols/server-decoration.xml" rel="noreferrer" target="_blank">https://github.com/KDE/kwayland/blob/master/src/client/protocols/server-decoration.xml</a><br>
> ><br>
> > \[1\] <a href="https://lists.freedesktop.org/archives/wayland-devel/2017-October/035564.html" rel="noreferrer" target="_blank">https://lists.freedesktop.org/archives/wayland-devel/2017-October/035564.html</a><br>
> ><br>
> > \[2\] <a href="https://github.com/swaywm/wlroots/pull/638" rel="noreferrer" target="_blank">https://github.com/swaywm/wlroots/pull/638</a><br>
> ><br>
> >  Makefile.am                                        |   1 +<br>
> ><br>
> >  unstable/xdg-toplevel-decoration/README            |   4 +<br>
> ><br>
> >  .../xdg-toplevel-decoration-unstable-v1.xml        | 127 +++++++++++++++++++++<br>
> ><br>
> >  3 files changed, 132 insertions(+)<br>
> ><br>
> >  create mode 100644 unstable/xdg-toplevel-decoration/README<br>
> ><br>
> >  create mode 100644 unstable/xdg-toplevel-decoration/xdg-toplevel-decoration-unstable-v1.xml<br>
> ><br>
> > diff --git a/Makefile.am b/Makefile.am<br>
> ><br>
> > index 4b9a901..07744e9 100644<br>
> ><br>
> > \-\-\- a/Makefile.am<br>
> ><br>
> > \+\+\+ b/Makefile.am<br>
> ><br>
> > @@ -17,6 +17,7 @@ unstable_protocols =                                                          \<br>
> ><br>
> >         unstable/keyboard-shortcuts-inhibit/keyboard-shortcuts-inhibit-unstable-v1.xml \<br>
> ><br>
> >         unstable/xdg-output/xdg-output-unstable-v1.xml                          \<br>
> ><br>
> >         unstable/input-timestamps/input-timestamps-unstable-v1.xml      \<br>
> ><br>
> > +       unstable/xdg-toplevel-decoration/xdg-toplevel-decoration-unstable-v1.xml        \<br>
> ><br>
> >         $(NULL)<br>
> ><br>
> >  stable_protocols =                                                             \<br>
> ><br>
> > diff --git a/unstable/xdg-toplevel-decoration/README b/unstable/xdg-toplevel-decoration/README<br>
> ><br>
> > new file mode 100644<br>
> ><br>
> > index 0000000..e927110<br>
> ><br>
> > \-\-\- /dev/null<br>
> ><br>
> > \+\+\+ b/unstable/xdg-toplevel-decoration/README<br>
> ><br>
> > @@ -0,0 +1,4 @@<br>
> ><br>
> > +xdg\_toplevel\_decoration protocol<br>
> ><br>
> > +<br>
> ><br>
> > +Maintainers:<br>
> ><br>
> > +Simon Ser <<a href="mailto:contact@emersion.fr" target="_blank">contact@emersion.fr</a>><br>
> ><br>
> > diff --git a/unstable/xdg-toplevel-decoration/xdg-toplevel-decoration-unstable-v1.xml b/unstable/xdg-toplevel-decoration/xdg-toplevel-decoration-unstable-v1.xml<br>
> ><br>
> > new file mode 100644<br>
> ><br>
> > index 0000000..acc1ed2<br>
> ><br>
> > \-\-\- /dev/null<br>
> ><br>
> > \+\+\+ b/unstable/xdg-toplevel-decoration/xdg-toplevel-decoration-unstable-v1.xml<br>
> ><br>
> > @@ -0,0 +1,127 @@<br>
> ><br>
> > +<?xml version="1.0" encoding="UTF-8"?><br>
> ><br>
> > +<protocol name="xdg\_toplevel\_decoration\_unstable\_v1"><br>
> ><br>
> > +  <copyright><br>
> ><br>
> > +    Copyright © 2018 Simon Ser<br>
> ><br>
> > +<br>
> ><br>
> > +    Permission is hereby granted, free of charge, to any person obtaining a<br>
> ><br>
> > +    copy of this software and associated documentation files (the "Software"),<br>
> ><br>
> > +    to deal in the Software without restriction, including without limitation<br>
> ><br>
> > +    the rights to use, copy, modify, merge, publish, distribute, sublicense,<br>
> ><br>
> > +    and/or sell copies of the Software, and to permit persons to whom the<br>
> ><br>
> > +    Software is furnished to do so, subject to the following conditions:<br>
> ><br>
> > +<br>
> ><br>
> > +    The above copyright notice and this permission notice (including the next<br>
> ><br>
> > +    paragraph) shall be included in all copies or substantial portions of the<br>
> ><br>
> > +    Software.<br>
> ><br>
> > +<br>
> ><br>
> > +    THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR<br>
> ><br>
> > +    IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,<br>
> ><br>
> > +    FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL<br>
> ><br>
> > +    THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER<br>
> ><br>
> > +    LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING<br>
> ><br>
> > +    FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER<br>
> ><br>
> > +    DEALINGS IN THE SOFTWARE.<br>
> ><br>
> > +  </copyright><br>
> ><br>
> > +<br>
> ><br>
> > +  <interface name="zxdg\_toplevel\_decoration\_manager\_v1" version="1"><br>
> ><br>
> > +    <description summary="window decoration manager"><br>
> ><br>
> > +      This interface permits choosing between client-side and server-side<br>
> ><br>
> > +      window decorations for a toplevel surface.<br>
> ><br>
> > +<br>
> ><br>
> > +      A window decoration is a user interface component used to move, resize and<br>
> ><br>
> > +      change a window's state. It can be managed either by the client (part of<br>
> ><br>
> > +      the surface) or by the server.<br>
> ><br>
> > +<br>
> ><br>
> > +      By advertizing this interface the server anounces support for server-side<br>
> ><br>
> > +      window decorations.<br>
> ><br>
> > +<br>
> ><br>
> > +      Warning! The protocol described in this file is experimental and<br>
> ><br>
> > +      backward incompatible changes may be made. Backward compatible changes<br>
> ><br>
> > +      may be added together with the corresponding interface version bump.<br>
> ><br>
> > +      Backward incompatible changes are done by bumping the version number in<br>
> ><br>
> > +      the protocol and interface names and resetting the interface version.<br>
> ><br>
> > +      Once the protocol is to be declared stable, the 'z' prefix and the<br>
> ><br>
> > +      version number in the protocol and interface names are removed and the<br>
> ><br>
> > +      interface version number is reset.<br>
> ><br>
> > +    </description><br>
> ><br>
> > +<br>
> ><br>
> > +    <enum name="error"><br>
> ><br>
> > +      <entry name="unconfigured_buffer" value="1"/><br>
> ><br>
> > +      <entry name="already_constructed" value="2"/><br>
> ><br>
> > +    </enum><br>
> ><br>
> > +<br>
> ><br>
> > +    <request name="destroy" type="destructor"><br>
> ><br>
> > +      <description summary="destroy the decoration manager object"><br>
> ><br>
> > +        Destroy the decoration manager.<br>
> ><br>
> > +      </description><br>
> ><br>
> > +    </request><br>
> ><br>
> > +<br>
> ><br>
> > +    <request name="get_decoration"><br>
> ><br>
> > +      <description summary="create a new decoration object"><br>
> ><br>
> > +        Create a new decoration object associated with the given toplevel.<br>
> ><br>
> > +<br>
> ><br>
> > +        Creating a zxdg\_toplevel\_decoration\_v1 from a xdg\_toplevel which has a<br>
> ><br>
> > +        buffer attached or committed is a client error, and any attempts by a<br>
> ><br>
> > +        client to attach or manipulate a buffer prior to the first<br>
> ><br>
> > +        zxdg\_toplevel\_decoration_v1.configure call must also be treated as<br>
> ><br>
> > +        errors.<br>
> ><br>
> > +      </description><br>
> ><br>
> > +      <arg name="id" type="new\_id" interface="zxdg\_toplevel\_decoration\_v1"/><br>
> ><br>
> > +      <arg name="toplevel" type="object" interface="xdg_toplevel"/><br>
> ><br>
> > +    </request><br>
> ><br>
> > +  </interface><br>
> ><br>
> > +<br>
> ><br>
> > +  <interface name="zxdg\_toplevel\_decoration_v1" version="1"><br>
> ><br>
> > +    <description summary="decoration object for a toplevel surface"><br>
> ><br>
> > +      The decoration object allows the client to switch between a client-side<br>
> ><br>
> > +      and server-side window decoration for a toplevel surface.<br>
> ><br>
> > +    </description><br>
> ><br>
> > +<br>
> ><br>
> > +    <request name="destroy" type="destructor"><br>
> ><br>
> > +      <description summary="destroy the decoration object"><br>
> ><br>
> > +        Switch back to client-side-only window decoration mode.<br>
> ><br>
> > +      </description><br>
> ><br>
> > +    </request><br>
> ><br>
> > +<br>
> ><br>
> > +    <request name="set_mode"><br>
> ><br>
> > +      <description summary="set the decoration mode"><br>
> ><br>
> > +        Set the toplevel surface decoration mode.<br>
> ><br>
> > +<br>
> ><br>
> > +        After requesting a decoration mode, the compositor will respond by<br>
> ><br>
> > +        emitting a xdg_surface.configure event. The client should then update<br>
> ><br>
> > +        its content, drawing it with or without decorations depending on the<br>
> ><br>
> > +        received mode. The client must also acknowledge the configure when<br>
> ><br>
> > +        committing the new content (see xdg\_surface.ack\_configure).<br>
> ><br>
> > +<br>
> ><br>
> > +        The compositor can ignore this request.<br>
> ><br>
> > +      </description><br>
> ><br>
> > +      <arg name="mode" type="uint" enum="mode" summary="the decoration mode"/><br>
> ><br>
> > +    </request><br>
> ><br>
> > +<br>
> ><br>
> > +    <event name="preferred_mode"><br>
> ><br>
> > +      <description summary="advertise the server's preferred mode"><br>
> ><br>
> > +        The preferred_mode event describes the server's preferred decoration<br>
> ><br>
> > +        mode for this toplevel surface. The event is sent when binding to the<br>
> ><br>
> > +        decoration object and whenever the preferred mode changes.<br>
> ><br>
> > +      </description><br>
> ><br>
> > +      <arg name="mode" type="uint" enum="mode" summary="the preferred mode"/><br>
> ><br>
> > +    </event><br>
> ><br>
> > +<br>
> ><br>
> > +    <event name="configure"><br>
> ><br>
> > +      <description summary="suggest a surface change"><br>
> ><br>
> > +        The configure event asks the client to change its decoration mode. The<br>
> ><br>
> > +        configured state should not be applied immediately. See<br>
> ><br>
> > +        xdg_surface.configure for details.<br>
> ><br>
> > +      </description><br>
> ><br>
> > +      <arg name="mode" type="uint" enum="mode" summary="the decoration mode"/><br>
> ><br>
> > +    </event><br>
> ><br>
> > +<br>
> ><br>
> > +    <enum name="mode"><br>
> ><br>
> > +      <description summary="window decoration modes"><br>
> ><br>
> > +        These values describe window decoration modes.<br>
> ><br>
> > +      </description><br>
> ><br>
> > +      <entry name="client" value="1" summary="client-side window decoration"/><br>
> ><br>
> > +      <entry name="server" value="2" summary="server-side window decoration"/><br>
> ><br>
> > +    </enum><br>
> ><br>
> > +  </interface><br>
> ><br>
> > +</protocol><br>
> ><br>
> > --<br>
> ><br>
> > 2.16.1<br>
> ><br>
> > \_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_<br>
> ><br>
> > wayland-devel mailing list<br>
> ><br>
> > <a href="mailto:wayland-devel@lists.freedesktop.org" target="_blank">wayland-devel@lists.freedesktop.org</a><br>
> ><br>
> > <a href="https://lists.freedesktop.org/mailman/listinfo/wayland-devel" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/wayland-devel</a><br>
<br>
<br>
</blockquote></div></div>