https://bugs.kde.org/show_bug.cgi?id=389117

Martin Flöser <mgraess...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
   Version Fixed In|                            |5.12.0
             Status|UNCONFIRMED                 |RESOLVED
      Latest Commit|                            |https://commits.kde.org/kwi
                   |                            |n/339a08de3883cc796a263183a
                   |                            |336a9e9cfaf4106

--- Comment #9 from Martin Flöser <mgraess...@kde.org> ---
Git commit 339a08de3883cc796a263183a336a9e9cfaf4106 by Martin Flöser.
Committed on 23/01/2018 at 15:58.
Pushed by graesslin into branch 'Plasma/5.12'.

Acknowledge the server side deco mode in WaylandServer not ShellClient

Summary:
With GTK applications our code showed issues. With GTK the sequence for
a menu or dialog window is:
 1. create surface
 2. create server side decoration
 3. request csd
 4. create xdg_shell_surface

Which result in KWin to have the ServerSideDecoration object being
created before the ShellClient is created. The connect for mode changed
happens in ShellClient, which is too late to catch that the window
requested client side decoration. Thus KWin created a decoration about
things such as a menu.

This change moves the acknowledge of the mode to WaylandServer. As we
anyway always acknowledge the requested mode we can also do it there and
don't require ShellClient to be present. Thus when the ShellClient is
created the mode is properly reflected and no decoration is created.
FIXED-IN: 5.12.0

Test Plan: Played a lot with evince

Reviewers: #kwin, #plasma

Subscribers: plasma-devel, kwin

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D9967

M  +0    -2    shell_client.cpp
M  +6    -0    wayland_server.cpp

https://commits.kde.org/kwin/339a08de3883cc796a263183a336a9e9cfaf4106

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to