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

List:       kde-commits
Subject:    KDE/kdegraphics/kolourpaint
From:       Christoph Feck <christoph () maxiom ! de>
Date:       2010-08-29 18:05:40
Message-ID: 20100829180540.38D05AC857 () svn ! kde ! org
[Download RAW message or body]

SVN commit 1169574 by cfeck:

Fix resizing grip handles causing X11 size overflow

Instead of sizing the handles along the full extent of the view,
we just size them small enough for the handle to be visible. The
mask is now actually redundant, but I do not feel like rewriting
it.

Tested with 1600% zoom on 12000x9000 image, as well as image
smaller than grip size.

CCBUG: 237936


 M  +4 -4      kpViewScrollableContainer.cpp  


--- trunk/KDE/kdegraphics/kolourpaint/kpViewScrollableContainer.cpp #1169573:1169574
@@ -1100,11 +1100,11 @@
 
     if (m_view)
     {
-        m_bottomGrip->setFixedWidth (m_view->width ());
-        moveChild (m_bottomGrip, 0, m_view->height ());
+        m_bottomGrip->setFixedWidth (kpGrip::Size);
+        moveChild (m_bottomGrip, (m_view->width () - m_bottomGrip->width ()) / 2, m_view->height ());
 
-        m_rightGrip->setFixedHeight (m_view->height ());
-        moveChild (m_rightGrip, m_view->width (), 0);
+        m_rightGrip->setFixedHeight (kpGrip::Size);
+        moveChild (m_rightGrip, m_view->width (), (m_view->height () - m_bottomGrip->height ()) / 2);
 
         moveChild (m_bottomRightGrip, m_view->width (), m_view->height ());
     #if DEBUG_KP_VIEW_SCROLLABLE_CONTAINER
[prev in list] [next in list] [prev in thread] [next in thread]