|Summary:||maximized windows have shadow|
|Product:||[Plasma] Oxygen||Reporter:||Antonio Rojas <arojas>|
|Component:||win deco||Assignee:||Hugo Pereira Da Costa <hugo.pereira.da.costa>|
|Severity:||normal||CC:||hugo.pereira.da.costa, KaiUweBroulik2, kwin-bugs-null|
|Latest Commit:||Version Fixed In:|
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.