Ensure top of QgsCollapsibleGroupBox is visible after expand. Previous

behaviour was to scroll to center of group box for long group boxes, which is confusing for users.
qgis · Sep 3, 2014 · 7819949 · 7819949
1 parent 224411b
commit 7819949
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/src/gui/qgscollapsiblegroupbox.cpp b/src/gui/qgscollapsiblegroupbox.cpp
@@ -434,7 +434,12 @@ void QgsCollapsibleGroupBoxBasic::setCollapsed( bool collapse )
   {
     // process events so entire widget is shown
     QApplication::processEvents();
+    mParentScrollArea->setUpdatesEnabled( false );
     mParentScrollArea->ensureWidgetVisible( this );
+    //and then make sure the top of the widget is visible - otherwise tall group boxes
+    //scroll to their centres, which is disorienting for users
+    mParentScrollArea->ensureWidgetVisible( mCollapseButton, 0, 5 );
+    mParentScrollArea->setUpdatesEnabled( true );
   }
   // emit signal for connections using collapsed state
   emit collapsedStateChanged( isCollapsed() );
@@ -447,7 +452,7 @@ void QgsCollapsibleGroupBoxBasic::collapseExpandFixes()
 
   if ( mCollapsed )
   {
-    Q_FOREACH ( QObject* child, children() )
+    Q_FOREACH( QObject* child, children() )
     {
       QWidget* w = qobject_cast<QWidget*>( child );
       if ( w && w != mCollapseButton )
@@ -459,7 +464,7 @@ void QgsCollapsibleGroupBoxBasic::collapseExpandFixes()
   }
   else // on expand
   {
-    Q_FOREACH ( QObject* child, children() )
+    Q_FOREACH( QObject* child, children() )
     {
       QWidget* w = qobject_cast<QWidget*>( child );
       if ( w && w != mCollapseButton )