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

List:       kde-commits
Subject:    KDE/kdelibs/kdeui/windowmanagement
From:       Aaron J. Seigo <aseigo () kde ! org>
Date:       2010-07-26 17:04:03
Message-ID: 20100726170403.DFC41AC7CA () svn ! kde ! org
[Download RAW message or body]

SVN commit 1154998 by aseigo:

since KWindowSystem::currentDesktop() always returns 1 on MS Windows, then it only \
makes sense for KWindowInfo::desktop() to also return 1 so that things like \
"windowInfo.isOnDesktop(KWindowSystem::currentDesktop())" actually stand a chance of \
working. if there's a reason for the mismatch, please let me know.

CCMAIL:kde-windows@kde.org


 M  +2 -4      kwindowinfo_win.cpp  


--- trunk/KDE/kdelibs/kdeui/windowmanagement/kwindowinfo_win.cpp #1154997:1154998
@@ -202,9 +202,7 @@
 
 bool KWindowInfo::isOnDesktop( int desk ) const
 {
-    if(desk == desktop())
-        return true;
-    return false;
+    return desk == desktop();
 }
 
 bool KWindowInfo::onAllDesktops() const
@@ -214,7 +212,7 @@
 
 int KWindowInfo::desktop() const
 {
-    return 0;
+    return 1;
 }
 
 QRect KWindowInfo::geometry() const


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