src/hbwidgets/itemviews/hbgridview.cpp
changeset 34 ed14f46c0e55
parent 6 c3690ec91ef8
--- a/src/hbwidgets/itemviews/hbgridview.cpp	Mon Oct 04 17:49:30 2010 +0300
+++ b/src/hbwidgets/itemviews/hbgridview.cpp	Mon Oct 18 18:23:13 2010 +0300
@@ -120,11 +120,13 @@
 {
     Q_D(HbGridView);
 
-    d->mVisibleIndex = d->indexInTheCenter();
+    if (d->itemContainer()->rowCount() != rowCount) {
+        d->mVisibleIndex = d->firstFullyVisibleIndex();
 
-    d->itemContainer()->setRowCount(rowCount);
+        d->itemContainer()->setRowCount(rowCount);
 
-    scrollTo(d->mVisibleIndex, HbAbstractItemView::PositionAtCenter);
+        scrollTo(d->mVisibleIndex, HbAbstractItemView::PositionAtTop);
+    }
 }
 
 /*!
@@ -147,11 +149,13 @@
 {
     Q_D(HbGridView);
 
-    d->mVisibleIndex = d->indexInTheCenter();
+    if (d->itemContainer()->columnCount() != columnCount) {
+        d->mVisibleIndex = d->firstFullyVisibleIndex();
 
-    d->itemContainer()->setColumnCount(columnCount);
+        d->itemContainer()->setColumnCount(columnCount);
 
-    scrollTo(d->mVisibleIndex, HbAbstractItemView::PositionAtCenter);
+        scrollTo(d->mVisibleIndex, HbAbstractItemView::PositionAtTop);
+    }
 }
 
 
@@ -252,12 +256,13 @@
 void HbGridView::scrollTo(const QModelIndex &index, ScrollHint hint)
 {
     Q_D(HbGridView);
-    // always use container, event if recycling is off and all items are 
-    // in container, but still additional action is needed -
+
+    // always use HbGridItemContainer::scrollTo, even if recycling
+    // is off - still additional action is needed -
     // container::scrollTo is responsible for procesing all
     // posponed events (DelayedLayoutRequest)
-    if (    d->mModelIterator->model()
-        &&  index.model() == d->mModelIterator->model()) {
+    if (   d->mModelIterator->model()
+        && index.model() == d->mModelIterator->model()) {
         d->itemContainer()->scrollTo(index, hint);
         HbAbstractItemView::scrollTo(index, hint);
     }
@@ -279,7 +284,7 @@
 void HbGridView::orientationAboutToBeChanged()
 {
     Q_D(HbGridView);
-    d->mVisibleIndex = d->indexInTheCenter();
+    d->mVisibleIndex = d->firstFullyVisibleIndex();
 }
 
 /*!
@@ -291,10 +296,20 @@
     d->mContainer->setPos(0,0);
     if (d->mSwapDimensionsOnOrientationChange) {
         d->itemContainer()->orientationChanged(newOrientation);
+        int indexPosition = d->mModelIterator->indexPosition(d->mVisibleIndex);
+        int columnsCount = 0;
+        if (d->mScrollDirections == Qt::Vertical) {
+            columnsCount = columnCount();
+        } else {
+            columnsCount = rowCount();
+        }
+        int remainder = indexPosition % columnsCount;
+        if (remainder > columnsCount / 2) {
+            d->mVisibleIndex = d->mModelIterator->index(indexPosition + columnsCount - remainder);
+        }
     }
 
-    // abstract part is enough - container update buffer
-    HbAbstractItemView::scrollTo(d->mVisibleIndex, HbAbstractItemView::PositionAtCenter);
+    scrollTo(d->mVisibleIndex, HbAbstractItemView::PositionAtTop);
 
     d->mVisibleIndex = QModelIndex();
 }
@@ -395,13 +410,12 @@
 {
     Q_D(HbGridView);
     // scroll direction changed, calculations need to be done on old value
-    d->mVisibleIndex = d->indexInTheCenter((d->mScrollDirections == Qt::Vertical) 
-        ? Qt::Horizontal : Qt::Vertical);
+    d->mVisibleIndex = d->firstFullyVisibleIndex();
     d->mContainer->setPos(0,0);
     d->itemContainer()->scrollDirectionChanged(scrollDirection);
 
     // abstract part is enough - container update buffer
-    HbAbstractItemView::scrollTo(d->mVisibleIndex, HbAbstractItemView::PositionAtCenter);
+    HbAbstractItemView::scrollTo(d->mVisibleIndex, HbAbstractItemView::PositionAtTop);
 
     d->mVisibleIndex = QModelIndex();
 }