Summary: | Logout effect completely broken | ||
---|---|---|---|
Product: | [Plasma] kwin | Reporter: | Martin Flöser <mgraesslin> |
Component: | effects-various | Assignee: | Martin Flöser <mgraesslin> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | kwin-bugs-null, mail, muziofg, rdieter |
Priority: | NOR | Flags: | mgraesslin:
ReviewRequest+
|
Version: | 4.8.90 | ||
Target Milestone: | 4.9 RC 2 | ||
Platform: | unspecified | ||
OS: | Linux | ||
URL: | https://git.reviewboard.kde.org/r/105459/ | ||
Latest Commit: | http://commits.kde.org/kde-workspace/2697a2732c8bebd0ffaa95ba6438ea27f37e3dcc | Version Fixed In: | 4.9.0 |
Description
Martin Flöser
2012-07-06 08:52:17 UTC
Just thrown a quick look on it. FTR: a) xrender /can/ paint a vignette (XRenderCreateRadialGradient, i'll write the part if you want) but it can NOT "do a stronger desaturation" - it actually can't desaturate at all. it can possibly have a blur as well (since it's not gaussian anyway) so we'll likely need no special casing here at all. b) i'd stay away from testing ksmserver class (the dialog could someday move to plasma or kded or whatever - actually having the ksmserver daemon link GUI classes only to show that dialog when it quits looks like a design flaw to me) c) to get it active it should be sufficient to set progress to 0.01 in slotWindowAdded > c) to get it active it should be sufficient to set progress to 0.01 in
> slotWindowAdded
I have most things already fixed here. For c it's a simple || logoutWindow as
logoutWindow is only not null in case the effect is active.
Git commit 2697a2732c8bebd0ffaa95ba6438ea27f37e3dcc by Martin Gräßlin. Committed on 09/07/2012 at 19:04. Pushed by graesslin into branch 'KDE/4.9'. Fix logout effect including port to OpenGL 2 The logout effect was rather broken. First of all it was excluded from build if OpenGL ES is present at build time. The reason for this is that the effect did not work with GLES. In order to fix that the vignetting is ported over to OpenGL 2 by using a dedicated shader. As well the lod based blur is added through a dedicated shader and uses framebuffer blit to get the current rendered buffer before rendering the logout window into a texture. Last but not least the isActive method was broken and is fixed by checking whether the logout window is around. FIXED-IN: 4.9.0 REVIEW: 105459 M +0 -3 kwin/effects/CMakeLists.txt M +6 -0 kwin/effects/logout/CMakeLists.txt A +9 -0 kwin/effects/logout/data/logout-blur.frag A +8 -0 kwin/effects/logout/data/vignetting.frag M +127 -58 kwin/effects/logout/logout.cpp M +5 -0 kwin/effects/logout/logout.h http://commits.kde.org/kde-workspace/2697a2732c8bebd0ffaa95ba6438ea27f37e3dcc *** Bug 304239 has been marked as a duplicate of this bug. *** |