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

List:       kde-commits
Subject:    KDE/kdepim/kdgantt
From:       Dag Andersen <danders () get2net ! dk>
Date:       2008-02-19 12:15:42
Message-ID: 1203423342.307528.15638.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 777023 by danders:

Think I cracked the problem with always getting the
vertical scrollbar because the viewport size was 2 pixels to big.
QGraphicsView uses maximumViewportSize() when calculating scrollbars
so we do the same. 
CCMAIL: steffen@kdab.net


 M  +4 -4      kdganttgraphicsview.cpp  


--- trunk/KDE/kdepim/kdgantt/kdganttgraphicsview.cpp #777022:777023
@@ -519,13 +519,13 @@
     r.setLeft( qMin( 0.0, r.left() ) );
     // TODO: take scrollbars into account (if not always on)
     // The scene should be at least the size of the viewport
-    QSizeF size = viewport()->size();
-    //TODO: why -2 below? size should be ex. frames etc?
+    // NOTE: scrollbar calculus uses maximumViewportSize() - frameWidth()*2 (not the \
same as viewport()->size()) +    QSizeF size = maximumViewportSize() - QSizeF( \
frameWidth()*2, frameWidth()*2 );  if ( size.width() > r.width() ) {
-        r.setWidth( size.width() - 2 ); 
+        r.setWidth( size.width() );
     }
     if ( size.height() > r.height() ) {
-        r.setHeight( size.height() - 2 );
+        r.setHeight( size.height() );
     }
     scene()->setSceneRect( r );
     BASE::resizeEvent( ev );


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