Bug 274177

Summary: maximized windows have shadow
Product: [Plasma] Oxygen Reporter: Antonio Rojas <arojas>
Component: win decoAssignee: Hugo Pereira Da Costa <hugo.pereira.da.costa>
Status: RESOLVED FIXED    
Severity: normal CC: hugo.pereira.da.costa, KaiUweBroulik2, kwin-bugs-null
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Arch Linux   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: unmaximized window
maximized window
full desktop

Description Antonio Rojas 2011-05-26 10:23:11 UTC
Version:           unspecified (using Devel) 
OS:                Linux

Using KDE 4.7 beta 1.
Shadows of maximized windows are visible behind the panel. This doesn't look good, maximized windows shouldn't have shadows.

Reproducible: Always
Comment 1 Martin Flöser 2011-05-27 16:13:30 UTC
Could you please provide a screenshot? My windows don't do that.
Comment 2 Antonio Rojas 2011-05-27 18:15:29 UTC
Created attachment 60390 [details]
unmaximized window
Comment 3 Antonio Rojas 2011-05-27 18:16:27 UTC
Created attachment 60391 [details]
maximized window

notice the red glow visible behind the panel
Comment 4 Martin Flöser 2011-05-27 18:54:15 UTC
which window decoration are you using? Could you please provide a screenshot of the complete desktop, that would help to identify anything "strange".
Comment 5 Antonio Rojas 2011-05-27 19:21:18 UTC
Created attachment 60399 [details]
full desktop

I'm using oxygen (with ridiculously large shadows to make the bug more obvious)
Comment 6 Thomas Lübking 2011-05-30 00:54:06 UTC
*** Bug 274470 has been marked as a duplicate of this bug. ***
Comment 7 Hugo Pereira Da Costa 2011-05-30 00:56:58 UTC
@Martin
Should this be dealt with by Oxygen, or by KWin ? 
(I would have expected kwin, since oxygen would need to change its metrics etc. on maximize state in order not to do that) ... 

And if kwin ? Then should be re-assigned, I guess ...
Comment 8 Martin Flöser 2011-05-30 16:44:41 UTC
I am not sure whether this can be handled in kwin correctly. Sure we can ignore the padding for maximized windows but it could look strange. I think it makes more sense to adjust the decorations to have 0 padding and adjusted rendering.
Comment 9 Hugo Pereira Da Costa 2011-05-30 16:50:18 UTC
Martin, 
I actually was thinking that the easiest is just ... Keep padding, etc. unchanged, and well, not to paint the shadow when the client is maximized.
Makes sense ? 
If you agree, I'll push and close.
Comment 10 Martin Flöser 2011-05-30 17:16:47 UTC
if possible try to adjust the padding calculations as having it just translucent means that we have 
to render more
Comment 11 Hugo Pereira Da Costa 2011-05-30 17:58:09 UTC
Git commit 6c2d50208a62bd070df06b3c7cfe95c75d6dd74e by Hugo Pereira Da Costa.
Committed on 30/05/2011 at 16:51.
Pushed by hpereiradacosta into branch 'master'.

do not render shadows when maximized.
CCBUG: 274177

M  +1    -1    kwin/clients/oxygen/oxygenclient.cpp     

http://commits.kde.org/kde-workspace/6c2d50208a62bd070df06b3c7cfe95c75d6dd74e
Comment 12 Hugo Pereira Da Costa 2011-05-30 17:58:09 UTC
Git commit 91d35f36372671d38d558a48df914adc87d134ab by Hugo Pereira Da Costa.
Committed on 30/05/2011 at 17:56.
Pushed by hpereiradacosta into branch 'master'.

Set padding to 0 when client is maximized.
CCBUG: 274177

M  +2    -1    kwin/clients/oxygen/oxygenclient.cpp     

http://commits.kde.org/kde-workspace/91d35f36372671d38d558a48df914adc87d134ab
Comment 13 Hugo Pereira Da Costa 2011-05-31 17:33:19 UTC
Commits above fix it. 
So closing.