From 17ea2bf146f2f35a929af42c977eb42a4f4f2147 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Martin=20Gr=C3=A4=C3=9Flin?= Date: Tue, 26 Jan 2010 15:26:48 +0000 Subject: [PATCH] setTabBoxWindow causes the ItemInfo to be regenerated, so we have to break the loop after finding a window. Makes more sense anyway as we do not have to go through the hash to find the item we already found. BUG: 222120 svn path=/trunk/KDE/kdebase/workspace/; revision=1080616 --- effects/boxswitch/boxswitch.cpp | 3 +++ 1 file changed, 3 insertions(+) diff --git a/effects/boxswitch/boxswitch.cpp b/effects/boxswitch/boxswitch.cpp index ba3ea860b0..12b48074da 100644 --- a/effects/boxswitch/boxswitch.cpp +++ b/effects/boxswitch/boxswitch.cpp @@ -257,7 +257,10 @@ void BoxSwitchEffect::windowInputMouseEvent( Window w, QEvent* e ) for( i = windows.constBegin(); i != windows.constEnd(); ++i ) { if( i.value()->clickable.contains( pos )) + { effects->setTabBoxWindow( i.key() ); + break; + } } // special handling for second half of window in case of animation and even number of windows if( mAnimateSwitch && ( windows.size() % 2 == 0 ) )