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

Vlad Zagorodniy <vladz...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REPORTED                    |RESOLVED
      Latest Commit|                            |https://commits.kde.org/kwi
                   |                            |n/fe4d69b653c67556ba1bc4964
                   |                            |28d981fdc003f4f
         Resolution|---                         |FIXED
   Version Fixed In|                            |5.15.0

--- Comment #9 from Vlad Zagorodniy <vladz...@gmail.com> ---
Git commit fe4d69b653c67556ba1bc496428d981fdc003f4f by Vlad Zagorodniy.
Committed on 16/10/2018 at 15:58.
Pushed by vladz into branch 'master'.

Keep Deleted transients above old parents

Summary:
If a modal window is closed, usually, it will go behind its parent. The
reason for this is that Workspace::constrainedStackingOrder() puts only
AbstractClient transients above parents, not Deleted transients.

So, if fade/glide/scale effect animates the disappearing of a transient,
unfortunately, one can't see that animation.
FIXED-IN: 5.15.0

Test Plan:
=== Closing of a transient and parent window

Before:

https://www.youtube.com/watch?v=XiLq7EAVCp0

After:

https://www.youtube.com/watch?v=cH_Ki-sqY8M

=== Scale effect

Before:

https://www.youtube.com/watch?v=Eb2a3U7R10I

After:

https://www.youtube.com/watch?v=4AKu3fdrnYQ

=== Sheet effect

Before:

https://www.youtube.com/watch?v=xPPSnR5FUU0

After:

https://www.youtube.com/watch?v=o_hxTNT-5Hg

=== Popup menus on Wayland

Before:

https://www.youtube.com/watch?v=5DnrY8p3F5A

After:

https://www.youtube.com/watch?v=7XEo8n_CrCc

Reviewers: #kwin, davidedmundson

Reviewed By: #kwin, davidedmundson

Subscribers: abetts, davidedmundson, kwin

Tags: #kwin

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

M  +217  -0    autotests/integration/stacking_order_test.cpp
M  +73   -0    deleted.cpp
M  +81   -0    deleted.h
M  +100  -19   layers.cpp
M  +1    -0    workspace.h

https://commits.kde.org/kwin/fe4d69b653c67556ba1bc496428d981fdc003f4f

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

Reply via email to