Fix layout issue for groups

When enabling a group where the sort-role is retrieved
asynchronously (e.g. group by "type") the group-headers have
stayed invisible.

diff --git a/src/kitemviews/kfileitemmodel.cpp b/src/kitemviews/kfileitemmodel.cpp
index 60f1c72ce4ebdf7342ea38b2b01d9121f4e65025..523fb619d766c1ede5bab4f7af9cd363e3fefc97 100644 (file)
--- a/src/kitemviews/kfileitemmodel.cpp
+++ b/src/kitemviews/kfileitemmodel.cpp
@@ -119,6 +119,9 @@ bool KFileItemModel::setData(int index, const QHash<QByteArray, QVariant>& value
         }
 
         if (!changedRoles.isEmpty()) {
         }
 
         if (!changedRoles.isEmpty()) {

+            if (groupedSorting() && changedRoles.contains(sortRole())) {
+                m_groups.clear();
+            }

             m_data[index] = currentValue;
             emit itemsChanged(KItemRangeList() << KItemRange(index, 1), changedRoles);
         }
             m_data[index] = currentValue;
             emit itemsChanged(KItemRangeList() << KItemRange(index, 1), changedRoles);
         }

diff --git a/src/kitemviews/kitemlistview.cpp b/src/kitemviews/kitemlistview.cpp
index 24fb62aefb921d20eecd1c8ea73ac3c6abb97171..b11fe905dbb8ae77af0ec36947741a044c1b8606 100644 (file)
--- a/src/kitemviews/kitemlistview.cpp
+++ b/src/kitemviews/kitemlistview.cpp
@@ -126,7 +126,7 @@ void KItemListView::setScrollOrientation(Qt::Orientation orientation)
         }
     }
 
         }
     }
 

-    updateLayout();
+    doLayout(Animation, 0, 0);

 
     onScrollOrientationChanged(orientation, previousOrientation);
     emit scrollOrientationChanged(orientation, previousOrientation);
 
     onScrollOrientationChanged(orientation, previousOrientation);
     emit scrollOrientationChanged(orientation, previousOrientation);


@@ -158,7 +158,7 @@ void KItemListView::setItemSize(const QSizeF& itemSize)
     }
 
     m_sizeHintResolver->clearCache();
     }
 
     m_sizeHintResolver->clearCache();

-    updateLayout();
+    doLayout(Animation, 0, 0);

     onItemSizeChanged(itemSize, previousSize);
 }
 
     onItemSizeChanged(itemSize, previousSize);
 }
 


@@ -182,7 +182,6 @@ void KItemListView::setScrollOffset(qreal offset)
     m_animation->setScrollOffset(offset);
     if (!m_layoutTimer->isActive()) {
         doLayout(NoAnimation, 0, 0);
     m_animation->setScrollOffset(offset);
     if (!m_layoutTimer->isActive()) {
         doLayout(NoAnimation, 0, 0);

-        update();

     }
     onScrollOffsetChanged(offset, previousOffset);
 }
     }
     onScrollOffsetChanged(offset, previousOffset);
 }


@@ -205,7 +204,6 @@ void KItemListView::setItemOffset(qreal offset)
     }
     if (!m_layoutTimer->isActive()) {
         doLayout(NoAnimation, 0, 0);
     }
     if (!m_layoutTimer->isActive()) {
         doLayout(NoAnimation, 0, 0);

-        update();

     }
 }
 
     }
 }
 


@@ -242,7 +240,7 @@ void KItemListView::setVisibleRoles(const QList<QByteArray>& roles)
     }
 
     updateVisibleRolesSizes();
     }
 
     updateVisibleRolesSizes();

-    updateLayout();
+    doLayout(Animation, 0, 0);

 
     onVisibleRolesChanged(roles, previousRoles);
 }
 
     onVisibleRolesChanged(roles, previousRoles);
 }


@@ -313,7 +311,7 @@ void KItemListView::setStyleOption(const KItemListStyleOption& option)
     }
 
     m_sizeHintResolver->clearCache();
     }
 
     m_sizeHintResolver->clearCache();

-    updateLayout();
+    doLayout(Animation, 0, 0);

     onStyleOptionChanged(option, previousOption);
 }
 
     onStyleOptionChanged(option, previousOption);
 }
 


@@ -432,7 +430,7 @@ void KItemListView::endTransaction()
 
     if (m_activeTransactions == 0) {
         onTransactionEnd();
 
     if (m_activeTransactions == 0) {
         onTransactionEnd();

-        updateLayout();
+        doLayout(Animation, 0, 0);

     }
 }
 
     }
 }
 


@@ -836,6 +834,7 @@ void KItemListView::slotItemsChanged(const KItemRangeList& itemRanges,
         if (updateSizeHints) {
             m_sizeHintResolver->itemsChanged(index, count, roles);
             m_layouter->markAsDirty();
         if (updateSizeHints) {
             m_sizeHintResolver->itemsChanged(index, count, roles);
             m_layouter->markAsDirty();

+

             if (!m_layoutTimer->isActive()) {
                 m_layoutTimer->start();
             }
             if (!m_layoutTimer->isActive()) {
                 m_layoutTimer->start();
             }


@@ -850,12 +849,22 @@ void KItemListView::slotItemsChanged(const KItemRangeList& itemRanges,
             }
         }
 
             }
         }
 

+        if (m_grouped && roles.contains(m_model->sortRole())) {
+            // The sort-role has been changed which might result
+            // in modified group headers
+            updateVisibleGroupHeaders();
+            doLayout(NoAnimation, 0, 0);
+            if (!m_layoutTimer->isActive()) {
+                m_layoutTimer->start();
+            }
+        }

     }
 }
 
 void KItemListView::slotGroupedSortingChanged(bool current)
 {
     m_grouped = current;
     }
 }
 
 void KItemListView::slotGroupedSortingChanged(bool current)
 {
     m_grouped = current;

+    m_layouter->markAsDirty();

 
     if (m_grouped) {
         // Apply the height of the header to the layouter
 
     if (m_grouped) {
         // Apply the height of the header to the layouter


@@ -863,10 +872,8 @@ void KItemListView::slotGroupedSortingChanged(bool current)
                                         m_styleOption.margin * 2;
         m_layouter->setGroupHeaderHeight(groupHeaderHeight);
 
                                         m_styleOption.margin * 2;
         m_layouter->setGroupHeaderHeight(groupHeaderHeight);
 

-        updateVisibleGroupHeaders(); // Triggers updateLayout()
+        updateVisibleGroupHeaders();

     } else {
     } else {

-        m_layouter->markAsDirty();
-

         // Clear all visible headers
         QMutableHashIterator<KItemListWidget*, KItemListGroupHeader*> it (m_visibleGroups);
         while (it.hasNext()) {
         // Clear all visible headers
         QMutableHashIterator<KItemListWidget*, KItemListGroupHeader*> it (m_visibleGroups);
         while (it.hasNext()) {


@@ -874,9 +881,9 @@ void KItemListView::slotGroupedSortingChanged(bool current)
             recycleGroupHeaderForWidget(it.key());
         }
         Q_ASSERT(m_visibleGroups.isEmpty());
             recycleGroupHeaderForWidget(it.key());
         }
         Q_ASSERT(m_visibleGroups.isEmpty());

-
-        updateLayout();

     }
     }

+
+    doLayout(Animation, 0, 0);

 }
 
 void KItemListView::slotSortOrderChanged(Qt::SortOrder current, Qt::SortOrder previous)
 }
 
 void KItemListView::slotSortOrderChanged(Qt::SortOrder current, Qt::SortOrder previous)


@@ -885,6 +892,7 @@ void KItemListView::slotSortOrderChanged(Qt::SortOrder current, Qt::SortOrder pr
     Q_UNUSED(previous);
     if (m_grouped) {
         updateVisibleGroupHeaders();
     Q_UNUSED(previous);
     if (m_grouped) {
         updateVisibleGroupHeaders();

+        doLayout(Animation, 0, 0);

     }
 }
 
     }
 }
 


@@ -894,6 +902,7 @@ void KItemListView::slotSortRoleChanged(const QByteArray& current, const QByteAr
     Q_UNUSED(previous);
     if (m_grouped) {
         updateVisibleGroupHeaders();
     Q_UNUSED(previous);
     if (m_grouped) {
         updateVisibleGroupHeaders();

+        doLayout(Animation, 0, 0);

     }
 }
 
     }
 }
 


@@ -1049,7 +1058,7 @@ void KItemListView::slotVisibleRoleWidthChanged(const QByteArray& role,
             widget->setVisibleRolesSizes(m_stretchedVisibleRolesSizes);
         }
 
             widget->setVisibleRolesSizes(m_stretchedVisibleRolesSizes);
         }
 

-        updateLayout();
+        doLayout(Animation, 0, 0);

     }
 }
 
     }
 }
 


@@ -1186,12 +1195,6 @@ KItemListRubberBand* KItemListView::rubberBand() const
     return m_rubberBand;
 }
 
     return m_rubberBand;
 }
 

-void KItemListView::updateLayout()
-{
-    doLayout(Animation, 0, 0);
-    update();
-}
-

 void KItemListView::doLayout(LayoutAnimationHint hint, int changedIndex, int changedCount)
 {
     if (m_layoutTimer->isActive()) {
 void KItemListView::doLayout(LayoutAnimationHint hint, int changedIndex, int changedCount)
 {
     if (m_layoutTimer->isActive()) {


@@ -1561,8 +1564,6 @@ void KItemListView::updateVisibleGroupHeaders()
         it.next();
         updateGroupHeaderForWidget(it.value());
     }
         it.next();
         updateGroupHeaderForWidget(it.value());
     }

-
-    updateLayout();

 }
 
 QHash<QByteArray, qreal> KItemListView::headerRolesWidths() const
 }
 
 QHash<QByteArray, qreal> KItemListView::headerRolesWidths() const

diff --git a/src/kitemviews/kitemlistview.h b/src/kitemviews/kitemlistview.h
index 397ecedf5764ba53fb02882ca99b1a57c80d7064..ff908000adcdb2051642cb4d9a89de2a2cf94eb9 100644 (file)
--- a/src/kitemviews/kitemlistview.h
+++ b/src/kitemviews/kitemlistview.h
@@ -301,7 +301,6 @@ private:
 
     KItemListRubberBand* rubberBand() const;
 
 
     KItemListRubberBand* rubberBand() const;
 

-    void updateLayout();

     void doLayout(LayoutAnimationHint hint, int changedIndex, int changedCount);
     void doGroupHeadersLayout(LayoutAnimationHint hint, int changedIndex, int changedCount);
     void emitOffsetChanges();
     void doLayout(LayoutAnimationHint hint, int changedIndex, int changedCount);
     void doGroupHeadersLayout(LayoutAnimationHint hint, int changedIndex, int changedCount);
     void emitOffsetChanges();