[PATCH] xdg-shell: require that popups intersect with parent surfaces

[Jonas Ådahl]

On Thu, Dec 08, 2016 at 11:00:10AM -0500, Mike Blumenkrantz wrote:
> some restrictions must be placed on this or else it becomes legal for
> the compositor to place popups in unexpected locations
> 
> Signed-off-by: Mike Blumenkrantz <zmike at osg.samsung.com>
> ---
>  unstable/xdg-shell/xdg-shell-unstable-v6.xml | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/unstable/xdg-shell/xdg-shell-unstable-v6.xml b/unstable/xdg-shell/xdg-shell-unstable-v6.xml
> index e49d74f..ab270b8 100644
> --- a/unstable/xdg-shell/xdg-shell-unstable-v6.xml
> +++ b/unstable/xdg-shell/xdg-shell-unstable-v6.xml
> @@ -118,7 +118,9 @@
>        child surface relative to a parent surface. Rules can be defined to ensure
>        the child surface remains within the visible area's borders, and to
>        specify how the child surface changes its position, such as sliding along
> -      an axis, or flipping around a rectangle.
> +      an axis, or flipping around a rectangle. These positioner-created rules are
> +      constrained by the requirement that a child surface must intersect with its
> +      parent surface.

I think it should be fine if they simply "touch" as in the intersection
is empty, but there is no space in between.

Otherwise I think it looks good.


Jonas

>  
>        See the various requests for details about possible rules.
>  
> @@ -941,7 +943,7 @@
>        The x and y arguments passed when creating the popup object specify
>        where the top left of the popup should be placed, relative to the
>        local surface coordinates of the parent surface. See
> -      xdg_surface.get_popup.
> +      xdg_surface.get_popup. An xdg_popup must intersect with its parent surface.
>  
>        The client must call wl_surface.commit on the corresponding wl_surface
>        for the xdg_popup state to take effect.
> -- 
> 2.5.5
> 
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/wayland-devel