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

List:       kde-commits
Subject:    extragear/utils/kpager
From:       Pino Toscano <pino () kde ! org>
Date:       2015-11-19 22:32:36
Message-ID: E1ZzXkq-0000Sw-04 () scm ! kde ! org
[Download RAW message or body]

SVN commit 1444727 by pino:

switch to QPoint for the last position


 M  +5 -8      kpager.cpp  
 M  +2 -5      kpager.kcfg  


--- trunk/extragear/utils/kpager/kpager.cpp #1444726:1444727
@@ -87,10 +87,9 @@
         resize(m_pPager->sizeHint());
     //  resize(cfg->readEntry(lWidth(),200),cfg->readEntry(lHeight(),90));
 
-    int xpos = Settings::xPos();
-    int ypos = Settings::yPos();
-    if (xpos > 0 && ypos > 0)
-      move(xpos,ypos);
+    const QPoint pos = Settings::lastPosition();
+    if (pos.x() > 0 && pos.y() > 0)
+      move(pos);
     else
     {
 //      NETRootInfo ri( QX11Info::display(), NET::WorkArea );
@@ -137,8 +136,7 @@
 
 bool KPagerMainWindow::queryClose()
 {
-    Settings::setXPos(x());
-    Settings::setYPos(y());
+    Settings::setLastPosition(pos());
     Settings::self()->findItem(m_pPager->lWidth())->setProperty(width());
     Settings::self()->findItem(m_pPager->lHeight())->setProperty(height());
     Settings::self()->writeConfig();
@@ -154,8 +152,7 @@
 
 bool KPagerMainWindow::queryExit()
 {
-    Settings::setXPos(x());
-    Settings::setYPos(y());
+    Settings::setLastPosition(pos());
     Settings::self()->findItem(m_pPager->lWidth())->setProperty(width());
     Settings::self()->findItem(m_pPager->lHeight())->setProperty(height());
     Settings::self()->writeConfig();
--- trunk/extragear/utils/kpager/kpager.kcfg #1444726:1444727
@@ -45,12 +45,9 @@
         <choice name="AllWindows" />
       </choices>
     </entry>
-    <entry key="xPos" type="Int">
-      <default>-1</default>
+    <entry key="lastPosition" type="Point">
+      <default>QPoint(-1, -1)</default>
     </entry>
-    <entry key="yPos" type="Int">
-      <default>-1</default>
-    </entry>
     <entry key="layoutClassicalWidth" type="Int">
       <default>-1</default>
     </entry>
[prev in list] [next in list] [prev in thread] [next in thread]