Bug 243890

Summary: Blur effect inteferes with kwin-drawn (not decoration) shadows
Product: [Plasma] kwin Reporter: Marcus Harrison <marcus>
Component: compositingAssignee: KWin default assignee <kwin-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: fredrik, grosser.meister.morti, jhnpwa, marc321, roland.leissa, whashnez, zerg2000-bugs.kde.org
Priority: HI    
Version: 0.2   
Target Milestone: ---   
Platform: Unlisted Binaries   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: screenshot of the bug in action

Description Marcus Harrison 2010-07-07 22:17:50 UTC
Version:           0.2 (using Devel) 
OS:                Linux

With the blur effect enabled, when new floating elements, such as tool-tips or menus, appear, their shadow is often not drawn. When a new floating element is created after, like a sub-menu, switching to another menu on the menubar or floating over another element that produces a tool-tip, the shadow is only drawn where the previous element's shadows would have covered. I have tested with and without the blur plugin active on various KWin effect settings, and it seems to be solely the blur effect's fault.

Reproducible: Always




OS: Linux (i686) release 2.6.32-24-generic
Compiler: cc
Comment 1 S. Christian Collins 2010-08-10 20:38:01 UTC
Created attachment 49976 [details]
screenshot of the bug in action
Comment 2 Martin Flöser 2010-08-10 20:59:21 UTC
*** Bug 244627 has been marked as a duplicate of this bug. ***
Comment 3 Bartosz Kwitniewski 2010-08-28 13:55:43 UTC
Same problem here.

Arch Linux x86_64
Nvidia 256.44 (GF 8800 GT)
KDE 4.5.0
Comment 4 whashnez 2010-08-31 08:47:07 UTC
I have also the same problem. How Bug 244627 is resolved? I did not see any solution.
Comment 5 Christoph Feck 2010-08-31 12:54:38 UTC
This bug is still open, and bug 244627 is the same bug, but we do not need two open bug reports for the same problem.

This is the reason one of them is closed/resolved as a "duplicate" of the other bug. Usually, the older bug stays, and the newer one is closed.
Comment 6 whashnez 2010-08-31 19:23:02 UTC
Not resolved in KDE 4.5.1
Comment 7 Thomas Lübking 2010-08-31 20:16:19 UTC
*** Bug 249662 has been marked as a duplicate of this bug. ***
Comment 8 whashnez 2010-10-06 10:32:55 UTC
Not resolved in KDE 4.5.2. Why?
Comment 9 Mathias Panzenböck 2010-11-06 18:51:03 UTC
Yep, still in in 4.5.2 Fedora 14 x86_64.
Comment 10 Law, Cheuk-wing 2010-11-27 15:14:12 UTC
The problem is still exist in KDE 4.5.3. But I found a workaround: Enable the "Scale In" effect.

Kubuntu 10.10 x86_64
Using the open source radeon driver (HD4850)
KDE 4.5.3
Comment 11 Martin Flöser 2011-01-31 22:06:20 UTC
Git commit 3b8984d630cc6dc655a171fa9c0f7e74e9ee3c61 by Martin Gr����lin.
Pushed by graesslin into branch 'graesslin/kwin-cleanup'.

Remove Shadow Effect.

The shadow effect is known to be broken since at least 4.5.
It is unfortunately in a state which makes it difficult to maintain
and the architecture has some serious drawbacks. Therefore it is the
best solution to replace the effect with a new and better
implementation. For more information about the new implementation
please see the discussion on KWin mailinglist:
http://lists.kde.org/?l=kwin&m=129607406517609&w=2

This also "fixes" all existing bug reports about the shadow effect.
Most of the bugs will really be fixed when the new shadow system is
implemented, if not it is a new bug and a new report should be created
for it.

Please excuse that we go this unnormal approach to mark bugs as
fixed with code removal.
BUG: 164084
BUG: 160948
BUG: 189241
BUG: 229164
BUG: 258663
BUG: 216709
BUG: 243890
FIXED-IN: 4.7.0

M  +0    -1    kwin/effects/CMakeLists.txt     
M  +0    -7    kwin/effects/configs_builtins.cpp     
D  +0    -31   kwin/effects/shadow/CMakeLists.txt     
D  +0    -786  kwin/effects/shadow/shadow.cpp     
D  +0    -166  kwin/effects/shadow/shadow.desktop     [TRAILING SPACE] [TRAILING SPACE]
D  +0    -102  kwin/effects/shadow/shadow.h     
D  +0    -128  kwin/effects/shadow/shadow_config.cpp     
D  +0    -91   kwin/effects/shadow/shadow_config.desktop     
D  +0    -57   kwin/effects/shadow/shadow_config.h     
D  +0    -258  kwin/effects/shadow/shadow_config.ui     
D  +0    -70   kwin/effects/shadow/shadow_helper.h     
M  +0    -4    kwin/kcmkwin/kwincompositing/main.cpp     
M  +9    -20   kwin/kcmkwin/kwincompositing/main.ui     

http://commits.kde.org/a5d5b61a/3b8984d630cc6dc655a171fa9c0f7e74e9ee3c61
Comment 12 whashnez 2011-02-01 11:00:16 UTC
That's good news, but for those who are impatient like me, a "workaround" is enabling the scale-in effect together with shadows and blur. This until the above is implemented.