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

David Edmundson <k...@davidedmundson.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |https://commits.kde.org/pla
                   |                            |sma-desktop/42a74c9a79baafb
                   |                            |222513f3ffad35b736b708454
             Status|CONFIRMED                   |RESOLVED
         Resolution|---                         |FIXED

--- Comment #12 from David Edmundson <k...@davidedmundson.co.uk> ---
Git commit 42a74c9a79baafb222513f3ffad35b736b708454 by David Edmundson.
Committed on 17/07/2018 at 09:34.
Pushed by davidedmundson into branch 'master'.

Fix blur behind folderview context menus

Summary:
Breeze has

if (!(widget->testAttribute(Qt::WA_WState_Created) ||
widget->internalWinId()))
   KWindowSystem::blur(widget->winId(), ...)

which makes sense as the style should not be creating surfaces in polish
and I don't want to remove it.

This is problematic with QMenu::polish which has a subtle behavioural
change compared to QMenu::exec in that it calls ensurePolish before
creating the surface.

Plasma::ContainmentInterface accidentally has this workaround already
for parent window setting.

Test Plan:
Right clicked on desktop icon
Looked amazing

Reviewers: #plasma, hein

Reviewed By: #plasma, hein

Subscribers: hein, plasma-devel

Tags: #plasma

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

M  +1    -0    containments/desktop/plugins/folder/foldermodel.cpp

https://commits.kde.org/plasma-desktop/42a74c9a79baafb222513f3ffad35b736b708454

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

Reply via email to