[PATCH wayland-protocols v5] unstable: add xdg-decoration protocol

Jonas Ådahl jadahl at gmail.com
Tue May 8 15:27:47 UTC 2018


On Tue, May 08, 2018 at 10:48:28AM -0400, Simon Ser wrote:
> Hi Jonas,
> 
> Any news about this trimmed-down version of the protocol?

Looks fine to me. Are all those Reviewed-by:s also for the trimmed down
version? Are going to propose a newer version or is the v5 of the patch
what should end up as version 1 of the extension?


Jonas

> 
> I'll reply to the v4 comments shortly.
> 
> Best,
> ---
> Simon Ser
> https://emersion.fr
> 
> ‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
> 
> On April 18, 2018 7:48 PM, Simon Ser <contact at emersion.fr> wrote:
> 
> >
> >
> > This adds a new protocol to negotiate server-side rendering of window
> >
> > decorations for xdg-toplevels. This allows compositors that want to draw
> >
> > decorations themselves to send their preference to clients, and clients that
> >
> > prefer server-side decorations to request them.
> >
> > This is inspired by a protocol from KDE [1] which has been implemented in
> >
> > KDE and Sway and was submitted for consideration in 2017 [2]. This patch
> >
> > provides an updated protocol with those concerns taken into account.
> >
> > Signed-off-by: Simon Ser contact at emersion.fr
> >
> > Reviewed-by: Drew DeVault sir at cmpwn.com
> >
> > Reviewed-by: David Edmundson david at davidedmundson.co.uk
> >
> > Reviewed-by: Alan Griffiths alan.griffiths at canonical.com
> >
> > Reviewed-by: Tony Crisci tony at dubstepdish.com
> >
> > Reviewed-by: Eike Hein hein at kde.org
> >
> > [1]https://github.com/KDE/kwayland/blob/master/src/client/protocols/server-decoration.xml
> >
> > [2] https://lists.freedesktop.org/archives/wayland-devel/2017-October/035564.html
> >
> >
> > --------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
> >
> > This was iterated on privately between representatives of Sway and wlroots
> >
> > (Simon Ser, Drew DeVault and Tony Crisci), KDE and Qt (David Edmundson), and
> >
> > Mir (Alan Griffiths).
> >
> > A proof-of-concept of a client and server implementation is available at [1].
> >
> > Changes from v4 to v5:
> >
> > -   Renamed the protocol to xdg-decoration for future extensions (Jonas)
> > -   Removed the preferred_mode event and the set_mode request for now because
> >
> >     of a race (Jonas)
> >
> >     [1] https://github.com/swaywm/wlroots/pull/638
> >
> >     Makefile.am | 1 +
> >
> >     unstable/xdg-decoration/README | 4 +
> >
> >     .../xdg-decoration-unstable-v1.xml | 115 ++++++++++++++++++
> >
> >     3 files changed, 120 insertions(+)
> >
> >     create mode 100644 unstable/xdg-decoration/README
> >
> >     create mode 100644 unstable/xdg-decoration/xdg-decoration-unstable-v1.xml
> >
> >     diff --git a/Makefile.am b/Makefile.am
> >
> >     index 4b9a901..71909d8 100644
> >
> >     --- a/Makefile.am
> >
> >     +++ b/Makefile.am
> >
> >     @@ -17,6 +17,7 @@ unstable_protocols = \
> >
> >     unstable/keyboard-shortcuts-inhibit/keyboard-shortcuts-inhibit-unstable-v1.xml \
> >
> >     unstable/xdg-output/xdg-output-unstable-v1.xml \
> >
> >     unstable/input-timestamps/input-timestamps-unstable-v1.xml \
> >
> >
> > -   unstable/xdg-decoration/xdg-decoration-unstable-v1.xml \
> >
> >     $(NULL)
> >
> >     stable_protocols = \
> >
> >     diff --git a/unstable/xdg-decoration/README b/unstable/xdg-decoration/README
> >
> >     new file mode 100644
> >
> >     index 0000000..e927110
> >
> >     --- /dev/null
> >
> >     +++ b/unstable/xdg-decoration/README
> >
> >     @@ -0,0 +1,4 @@
> >
> >     +xdg_toplevel_decoration protocol
> >
> > -
> >
> > +Maintainers:
> >
> > +Simon Ser contact at emersion.fr
> >
> > diff --git a/unstable/xdg-decoration/xdg-decoration-unstable-v1.xml b/unstable/xdg-decoration/xdg-decoration-unstable-v1.xml
> >
> > new file mode 100644
> >
> > index 0000000..935f1f5
> >
> > --- /dev/null
> >
> > +++ b/unstable/xdg-decoration/xdg-decoration-unstable-v1.xml
> >
> > @@ -0,0 +1,115 @@
> >
> > +<?xml version="1.0" encoding="UTF-8"?>
> >
> > +<protocol name="xdg_decoration_unstable_v1">
> >
> > -   <copyright>
> >
> > -   Copyright © 2018 Simon Ser
> >
> > -
> > -   Permission is hereby granted, free of charge, to any person obtaining a
> >
> > -   copy of this software and associated documentation files (the "Software"),
> >
> > -   to deal in the Software without restriction, including without limitation
> >
> > -   the rights to use, copy, modify, merge, publish, distribute, sublicense,
> >
> > -   and/or sell copies of the Software, and to permit persons to whom the
> >
> > -   Software is furnished to do so, subject to the following conditions:
> >
> > -
> > -   The above copyright notice and this permission notice (including the next
> >
> > -   paragraph) shall be included in all copies or substantial portions of the
> >
> > -   Software.
> >
> > -
> > -   THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> >
> > -   IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> >
> > -   FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
> >
> > -   THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> >
> > -   LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> >
> > -   FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
> >
> > -   DEALINGS IN THE SOFTWARE.
> >
> > -   </copyright>
> >
> > -
> > -   <interface name="zxdg_decoration_manager_v1" version="1">
> >
> > -   <description summary="window decoration manager">
> >
> > -        This interface allows a compositor to announce support for server-side
> >
> >
> > -        decorations and optionally express a preference for using them.
> >
> >
> > -
> > -        A window decoration is a set of window controls as deemed appropriate by
> >
> >
> > -        the party managing them, such as user interface components used to move,
> >
> >
> > -        resize and change a window's state.
> >
> >
> > -
> > -        A client can use this protocol to request being decorated by a supporting
> >
> >
> > -        compositor.
> >
> >
> > -
> > -        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.
> >
> >
> > -
> > -        Warning! The protocol described in this file is experimental and
> >
> >
> > -        backward incompatible changes may be made. Backward compatible changes
> >
> >
> > -        may be added together with the corresponding interface version bump.
> >
> >
> > -        Backward incompatible changes are done by bumping the version number in
> >
> >
> > -        the protocol and interface names and resetting the interface version.
> >
> >
> > -        Once the protocol is to be declared stable, the 'z' prefix and the
> >
> >
> > -        version number in the protocol and interface names are removed and the
> >
> >
> > -        interface version number is reset.
> >
> >
> > -   </description>
> >
> > -
> > -   <enum name="error">
> >
> > -        <entry name="unconfigured_buffer" value="1"/>
> >
> >
> > -        <entry name="already_constructed" value="2"/>
> >
> >
> > -   </enum>
> >
> > -
> > -   <request name="destroy" type="destructor">
> >
> > -        <description summary="destroy the decoration manager object">
> >
> >
> > -          Destroy the decoration manager.
> >
> >
> > -        </description>
> >
> >
> > -   </request>
> >
> > -
> > -   <request name="get_toplevel_decoration">
> >
> > -        <description summary="create a new toplevel decoration object">
> >
> >
> > -          Create a new decoration object associated with the given toplevel.
> >
> >
> > -
> > -          Creating an xdg_toplevel_decoration from an xdg_toplevel which has a
> >
> >
> > -          buffer attached or committed is a client error, and any attempts by a
> >
> >
> > -          client to attach or manipulate a buffer prior to the first
> >
> >
> > -          xdg_toplevel_decoration.configure call must also be treated as
> >
> >
> > -          errors.
> >
> >
> > -        </description>
> >
> >
> > -        <arg name="id" type="new_id" interface="zxdg_toplevel_decoration_v1"/>
> >
> >
> > -        <arg name="toplevel" type="object" interface="xdg_toplevel"/>
> >
> >
> > -   </request>
> >
> > -   </interface>
> >
> > -
> > -   <interface name="zxdg_toplevel_decoration_v1" version="1">
> >
> > -   <description summary="decoration object for a toplevel surface">
> >
> > -        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 xdg_toplevel_decoration object must be destroyed before its
> >
> >
> > -        xdg_toplevel.
> >
> >
> > -   </description>
> >
> > -
> > -   <request name="destroy" type="destructor">
> >
> > -        <description summary="destroy the decoration object">
> >
> >
> > -          Switch back to a mode without any server-side decorations at the next
> >
> >
> > -          commit.
> >
> >
> > -        </description>
> >
> >
> > -   </request>
> >
> > -
> > -   <enum name="mode">
> >
> > -        <description summary="window decoration modes">
> >
> >
> > -          These values describe window decoration modes.
> >
> >
> > -        </description>
> >
> >
> > -        <entry name="client_side" value="1" summary="no server-side window decoration"/>
> >
> >
> > -        <entry name="server_side" value="2" summary="server-side window decoration"/>
> >
> >
> > -   </enum>
> >
> > -
> > -   <event name="configure">
> >
> > -        <description summary="suggest a surface change">
> >
> >
> > -          The configure event asks the client to change its decoration mode. The
> >
> >
> > -          configured state should not be applied immediately. See
> >
> >
> > -          xdg_surface.configure for details.
> >
> >
> > -
> > -          A configure event can be sent at any time. The specified mode must be
> >
> >
> > -          obeyed by the client.
> >
> >
> > -        </description>
> >
> >
> > -        <arg name="mode" type="uint" enum="mode" summary="the decoration mode"/>
> >
> >
> > -   </event>
> >
> > -   </interface>
> >
> >
> > +</protocol>
> >
> > --
> >
> > 2.17.0
> >
> > wayland-devel mailing list
> >
> > wayland-devel at lists.freedesktop.org
> >
> > https://lists.freedesktop.org/mailman/listinfo/wayland-devel
> 
> 


More information about the wayland-devel mailing list