[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    KDE/kdebase/workspace/plasma/applets/pager
From:       Beat Wolf <asraniel () fryx ! ch>
Date:       2008-12-23 0:48:11
Message-ID: 1229993291.305859.27210.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 900492 by beatwolf:

don't show windows that are on all dekstops multiple times in popup
BUG:178518


 M  +6 -1      pager.cpp  


--- trunk/KDE/kdebase/workspace/plasma/applets/pager/pager.cpp #900491:900492
@@ -1122,8 +1122,12 @@
     QString subtext = QString();
     int taskCounter = 0;
     int displayedTaskCounter = 0;
+
+    QList<WId> windows;
+
     foreach(KWindowInfo winInfo, m_windowInfo){
-        if (winInfo.isOnDesktop(hoverDesktopNumber) || winInfo.onAllDesktops()) {
+        if ((winInfo.isOnDesktop(hoverDesktopNumber) || winInfo.onAllDesktops() )
+                                                    && !windows.contains(winInfo.win())) {
             bool active = (winInfo.win() == KWindowSystem::activeWindow());
             if ((taskCounter < 4) || active){    
                 QPixmap icon = KWindowSystem::icon(winInfo.win(), 16, 16, true);
@@ -1136,6 +1140,7 @@
                 subtext += (active ? "<u>" : "") + winInfo.visibleName() + (active ? "</u>" : "");
 
                 displayedTaskCounter++; 
+                windows.append(winInfo.win());
             }
             taskCounter++;
         }
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic