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

List:       kde-commits
Subject:    KDE/kdebase/workspace/plasma/plasma
From:       Marco Martin <notmart () gmail ! com>
Date:       2008-05-12 21:44:58
Message-ID: 1210628698.768323.3393.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 807087 by mart:

fized several panel sizing problems with center alignment:
fixed offset dragging
fixed positioning of the left vertical panel
fixed some 1 px skew when switching alignments


 M  +7 -7      panelview.cpp  


--- trunk/KDE/kdebase/workspace/plasma/plasma/panelview.cpp #807086:807087
@@ -149,13 +149,13 @@
                  m_offset = screenGeom.right() - geom.width();
              }
         } else if (m_alignment == Qt::AlignRight) {
-             if (screenGeom.right() - m_offset - geom.width() < 0 ) {
+             if (screenGeom.right() - m_offset - geom.width() < -1 ) {
                  m_offset = screenGeom.right() - geom.width();
              }
         } else if (m_alignment == Qt::AlignCenter) {
              if (screenGeom.center().x() + m_offset + geom.width()/2 > \
                screenGeom.right() + 1) {
                  m_offset = screenGeom.right() - geom.width()/2 - \
                screenGeom.center().x();
-             } else if (screenGeom.center().x() + m_offset - geom.width()/2 < 0) {
+             } else if (screenGeom.center().x() + m_offset - geom.width()/2 < -1) {
                  m_offset = screenGeom.center().x() - geom.width()/2;
              }
         }
@@ -178,13 +178,13 @@
                 m_offset = screenGeom.height() - geom.height();
             }
         } else if (m_alignment == Qt::AlignRight) {
-            if (screenGeom.bottom() - m_offset - geom.height() < 0) {
+            if (screenGeom.bottom() - m_offset - geom.height() < -1) {
                 m_offset = screenGeom.bottom() - geom.height();
             }
         } else if (m_alignment == Qt::AlignCenter) {
             if (screenGeom.center().y() + m_offset + geom.height()/2 > \
                screenGeom.bottom() + 1) {
                 m_offset = screenGeom.bottom() - geom.height()/2 - \
                screenGeom.center().y();
-             } else if (screenGeom.center().y() + m_offset - geom.width()/2 < 0) {
+             } else if (screenGeom.center().y() + m_offset - geom.width()/2 < -1) {
                 m_offset = screenGeom.center().y() - geom.width()/2;
              }
         }
@@ -216,7 +216,7 @@
         } else if (m_alignment == Qt::AlignRight) {
             geom.moveBottomLeft(QPoint(screenGeom.left(), screenGeom.bottom() - \
m_offset));  } else if (m_alignment == Qt::AlignCenter) {
-            geom.moveCenter(QPoint(screenGeom.left()+geom.height()/2, \
screenGeom.center().y() + m_offset)); +            \
geom.moveCenter(QPoint(screenGeom.left()+size.width()/2 - 1, screenGeom.center().y() \
+ m_offset -1));  }
 
         //enable borders if needed
@@ -229,7 +229,7 @@
         } else if (m_alignment == Qt::AlignRight) {
             geom.moveBottomLeft(QPoint(screenGeom.right() - size.width() + 1, \
screenGeom.bottom() - m_offset));  } else if (m_alignment == Qt::AlignCenter) {
-            geom.moveCenter(QPoint(screenGeom.right() - size.width()/2 + 1, \
screenGeom.center().y() + m_offset)); +            \
geom.moveCenter(QPoint(screenGeom.right() - size.width()/2, screenGeom.center().y() + \
m_offset));  }
 
         //enable borders if needed
@@ -243,7 +243,7 @@
         } else if (m_alignment == Qt::AlignRight) {
             geom.moveTopRight(QPoint(screenGeom.right() - m_offset, \
screenGeom.bottom() - size.height() + 1));  } else if (m_alignment == \
                Qt::AlignCenter) {
-            geom.moveCenter(QPoint(screenGeom.center().x() + m_offset, \
screenGeom.bottom() - size.height()/2 + 1)); +            \
geom.moveCenter(QPoint(screenGeom.center().x() + m_offset, screenGeom.bottom() - \
size.height()/2));  }
 
         //enable borders if needed


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

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