[PATCH wayland-protocols v3 5/7] xdg-shell: Make get_popup take a xdg_surface instead of wl_surface

Benoit Gschwind gschwind at gnu-log.net
Wed Jun 8 15:03:54 UTC 2016


Hello Jonas,

Fine for me.

Reviewed-by: Benoit Gschwind <gschwind at gnu-log.net>

Best regards

On 26/05/2016 06:32, Jonas Ådahl wrote:
> The reason for using wl_surface before was that xdg_popup and
> xdg_surface (now xdg_toplevel) had no common interface other than
> wl_surface, but since xdg_surface is now the base interface, lets use
> that.
> 
> Signed-off-by: Jonas Ådahl <jadahl at gmail.com>
> Reviewed-by: Mike Blumenkrantz <zmike at osg.samsung.com>
> Reviewed-by: Yong Bakos <ybakos at humanoriented.com>
> ---
>  unstable/xdg-shell/xdg-shell-unstable-v6.xml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/unstable/xdg-shell/xdg-shell-unstable-v6.xml b/unstable/xdg-shell/xdg-shell-unstable-v6.xml
> index 4a19c83..2fdcf9b 100644
> --- a/unstable/xdg-shell/xdg-shell-unstable-v6.xml
> +++ b/unstable/xdg-shell/xdg-shell-unstable-v6.xml
> @@ -165,7 +165,7 @@
>  	xdg_popup is and how it is used.
>        </description>
>        <arg name="id" type="new_id" interface="zxdg_popup_v6"/>
> -      <arg name="parent" type="object" interface="wl_surface"/>
> +      <arg name="parent" type="object" interface="zxdg_surface_v6"/>
>        <arg name="seat" type="object" interface="wl_seat" summary="the wl_seat of the user event"/>
>        <arg name="serial" type="uint" summary="the serial of the user event"/>
>        <arg name="x" type="int"/>
> 


More information about the wayland-devel mailing list