From 8fa22d0f6a6755a84c4bbbfe2511178da6290390 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Martin=20Gr=C3=A4=C3=9Flin?= Date: Sun, 26 Jun 2011 14:59:39 +0200 Subject: [PATCH] Remove PaintClipper from CoverSwitch Reflection in a vertical multiscreen setup are kind of broken without PaintClipper. But we have to ask ourselfe whether CoverSwitch in a vertical multi screen setup makes sense and whether it's useful to have reflections in such a setup. Horizontal multiscreen setup works fine (also with reflections). --- effects/coverswitch/coverswitch.cpp | 4 ---- 1 file changed, 4 deletions(-) diff --git a/effects/coverswitch/coverswitch.cpp b/effects/coverswitch/coverswitch.cpp index 352236033f..6bc98b3a02 100644 --- a/effects/coverswitch/coverswitch.cpp +++ b/effects/coverswitch/coverswitch.cpp @@ -260,14 +260,10 @@ void CoverSwitchEffect::paintScreen(int mask, QRegion region, ScreenPaintData& d } if (reflection) { - // restrict painting the reflections to the current screen - QRegion clip = QRegion(area); - PaintClipper::push(clip); // no reflections during start and stop animation // except when using a shader if ((!start && !stop) || ShaderManager::instance()->isValid()) paintScene(frontWindow, leftWindows, rightWindows, true); - PaintClipper::pop(clip); glEnable(GL_BLEND); glBlendFunc(GL_SRC_ALPHA, GL_ONE_MINUS_SRC_ALPHA); #ifndef KWIN_HAVE_OPENGLES