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

List:       kde-bugs-dist
Subject:    [Bug 109904] KMix slider appears on wrong twinview screen
From:       James Ots <kde () jamesots ! com>
Date:       2005-08-01 0:08:43
Message-ID: 20050801000843.28580.qmail () ktown ! kde ! org
[Download RAW message or body]

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
         
http://bugs.kde.org/show_bug.cgi?id=109904         
kde jamesots com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From kde jamesots com  2005-08-01 02:08 -------
Just fixed this bug. Needed vScreenSize.x() when calculating positions of multihead \
screens.

Sorry, forgot to put CCBUGS in the commit.

--- kmixdockwidget.cpp  (revision 441797)
+++ kmixdockwidget.cpp  (working copy)
 @ -264,12 +264,12  @
                // moved out-of-the screen on the right (see Bug 101742).
                QDesktopWidget* vdesktop = QApplication::desktop();
                const QRect& vScreenSize = vdesktop->screenGeometry(_dockAreaPopup);
-               if ( (x+_dockAreaPopup->width()) > (vScreenSize.width()) ) {
+               if ( (x+_dockAreaPopup->width()) > (vScreenSize.width() + \
                vScreenSize.x()) ) {
                        // move horizontally, so that it is completely visible
-                       _dockAreaPopup->move(vScreenSize.width() - \
_dockAreaPopup->width() -1 , y); +                       \
_dockAreaPopup->move(vScreenSize.width() + vScreenSize.x() - _dockAreaPopup->width() \
-1 , y);  } // horizontally out-of bound
-               else if ( x < 0 ) {
-                       _dockAreaPopup->move(0, y);
+               else if ( x < vScreenSize.x() ) {
+                       _dockAreaPopup->move(vScreenSize.x(), y);
                }
                // the above stuff could also be implemented vertically


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