[PATCH wayland-protocols v6 2/2] Add keyboard shortcuts inhibitor

Olivier Fourdan ofourdan at redhat.com
Tue Jun 6 09:55:22 UTC 2017




----- Original Message -----
> This adds a new protocol to let Wayland clients specify that they want
> all keyboard events to be sent to the client, regardless of the
> compositor own shortcuts.
> 
> This protocol can be used for virtual machine and remote connection
> viewers which require to pass all keyboard shortcuts to the hosted or
> remote system instead of being caught up by the compositor locally.
> 
> Signed-off-by: Olivier Fourdan <ofourdan at redhat.com>
> ---
>  v6: Indentation issues, rephrase "active" event description as per Peter's
>      review
>      Remove the key combo definition after discussing with Peter and Jonas
>      on irc #wayland:
>      https://people.freedesktop.org/~cbrill/dri-log/index.php?channel=wayland&date=2017-06-06
> 
>  Makefile.am                                        |   1 +
>  unstable/keyboard-shortcuts-inhibit/README         |   4 +
>  .../keyboard-shortcuts-inhibit-unstable-v1.xml     | 134
>  +++++++++++++++++++++
>  3 files changed, 139 insertions(+)
>  create mode 100644 unstable/keyboard-shortcuts-inhibit/README
>  create mode 100644
>  unstable/keyboard-shortcuts-inhibit/keyboard-shortcuts-inhibit-unstable-v1.xml
> 
> diff --git a/Makefile.am b/Makefile.am
> index 12465e6..d100c13 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -13,6 +13,7 @@ unstable_protocols =								\
>  	unstable/xdg-foreign/xdg-foreign-unstable-v1.xml			\
>  	unstable/idle-inhibit/idle-inhibit-unstable-v1.xml			\
>  	unstable/xwayland-keyboard-grab/xwayland-keyboard-grab-unstable-v1.xml	\
> +
> 	unstable/keyboard-shortcuts-inhibit/keyboard-shortcuts-inhibit-unstable-v1.xml
> \
>  	$(NULL)
>  
>  stable_protocols =								\
> diff --git a/unstable/keyboard-shortcuts-inhibit/README
> b/unstable/keyboard-shortcuts-inhibit/README
> new file mode 100644
> index 0000000..63ff335
> --- /dev/null
> +++ b/unstable/keyboard-shortcuts-inhibit/README
> @@ -0,0 +1,4 @@
> +Compositor shortcuts inhibit protocol
> +
> +Maintainers:
> +Olivier Fourdan <ofourdan at redhat.com>
> diff --git
> a/unstable/keyboard-shortcuts-inhibit/keyboard-shortcuts-inhibit-unstable-v1.xml
> b/unstable/keyboard-shortcuts-inhibit/keyboard-shortcuts-inhibit-unstable-v1.xml
> new file mode 100644
> index 0000000..65297f9
> --- /dev/null
> +++
> b/unstable/keyboard-shortcuts-inhibit/keyboard-shortcuts-inhibit-unstable-v1.xml
> @@ -0,0 +1,134 @@
> +<?xml version="1.0" encoding="UTF-8"?>
> +<protocol name="keyboard_shortcuts_inhibit_unstable_v1">
> +
> +  <copyright>
> +    Copyright © 2017 Red Hat Inc.
> +
> +    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>
> +
> +  <description summary="Protocol for inhibiting the compositor keyboard
> shortcuts">
> +    This protocol specifies a way for a client to request the compositor
> +    to ignore its own keyboard shortcuts for a given seat, so that all
> +    key events from that seat get forwarded to a surface.
> +
> +    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>
> +
> +  <interface name="zwp_keyboard_shortcuts_inhibit_manager_v1" version="1">
> +    <description summary="context object for keyboard grab_manager">
> +      A global interface used for inhibiting the compositor keyboard
> shortcuts.
> +    </description>
> +
> +    <request name="destroy" type="destructor">
> +      <description summary="destroy the keyboard shortcuts inhibitor
> object">
> +	Destroy the keyboard shortcuts inhibitor manager.
> +      </description>
> +    </request>
> +
> +    <request name="inhibit_shortcuts">
> +      <description summary="create a new keyboard shortcuts inhibitor
> object">
> +	Create a new keyboard shortcuts inhibitor object associated with
> +	the given surface for the given seat.
> +
> +	If shortcuts are already inhibited for the specified seat and surface,
> +	a protocol error "already_inhibited" is raised by the compositor.
> +      </description>
> +      <arg name="id" type="new_id"
> interface="zwp_keyboard_shortcuts_inhibitor_v1"/>
> +      <arg name="surface" type="object" interface="wl_surface"
> +	   summary="the surface that inhibits the keyboard shortcuts behavior"/>
> +      <arg name="seat" type="object" interface="wl_seat"
> +	   summary="the wl_seat for which keyboard shortcuts should be disabled"/>
> +    </request>
> +
> +  </interface>
> +
> +  <interface name="zwp_keyboard_shortcuts_inhibitor_v1" version="1">
> +    <description summary="context object for keyboard shortcuts inhibitor">
> +      A keyboard shortcuts inhibitor instructs the compositor to ignore
> +      its own keyboard shortcuts when the associated surface has keyboard
> +      focus. As a result, when the surface is focused, it will receive all
> +      key events originating from the specified seat, even those which
> +      would normally be caught by the compositor for its own shortcuts.
> +
> +      The Wayland compositor is however under no obligation to disable
> +      all of its shortcuts, and may keep some special key combo for its own
> +      use, including but not limited to one allowing the user to forcibly
> +      restore normal keyboard events routing in the case of an unwilling
> +      client. The compositor may also use the same key combo to reactivate
> +      an existing shortcut inhibitor that was previously deactivated on
> +      user request.
> +
> +      When the compositor restores its own keyboard shortcuts, an
> +      "inactive" event is emitted to notify the client that the keyboard
> +      shortcuts inhibitor is not effectively active for the surface and
> +      seat any more, and the client should not expect to receive all
> +      keyboard events.

Oh wait, there is no "inactive" event definition!

> +
> +      When the keyboard shortcuts inhibitor is inactive, the client has
> +      no way to forcibly reactivate the keyboard shortcuts inhibitor.
> +
> +      The user can chose to re-enable a previously deactivated keyboard
> +      shortcuts inhibitor using any mechanism the compositor may offer,
> +      in which case the compositor will send an "active" event to notify
> +      the client.
> +
> +      If the surface is destroyed, unmapped, or loses the seat's keyboard
> +      focus, the keyboard shortcuts inhibitor becomes irrelevant and the
> +      compositor will restore its own keyboard shortcuts but no "inactive"
> +      event is emitted in this case.
> +    </description>
> +
> +    <request name="destroy" type="destructor">
> +      <description summary="destroy the keyboard shortcuts inhibitor
> object">
> +	Remove the keyboard shortcuts inhibitor from the associated wl_surface.
> +      </description>
> +    </request>
> +
> +    <event name="active">
> +      <description summary="shortcuts are inhibited">
> +	This event indicates that the shortcut inhibitor is active.
> +
> +	The compositor sends this event every time the surface is given
> +	full access to all keyboard shortcuts, including the ones usually
> +	reserved by the compositors (see zwp_keyboard_shortcuts_inhibitor_v1
> +	for details though). This occurs typically when the initial request
> +	"inhibit_shortcuts" first becomes active or when the user instructs
> +	the compositor to re-enable and existing shortcuts inhibitor using
> +	any mechanism offered by the compositor.
> +      </description>
> +    </event>
> +
> +    <enum name="error">
> +      <entry name="already_inhibited"
> +	     value="0"
> +	     summary="the shortcuts are already inhibited for this surface"/>
> +    </enum>
> +  </interface>
> +</protocol>
> --
> 2.9.4
> 
> 


More information about the xorg-devel mailing list