Fix selection rect after porting from QFontMetrics::width()

[dolphin.git] / src / kitemviews / kitemlistviewaccessible.cpp

diff --git a/src/kitemviews/kitemlistviewaccessible.cpp b/src/kitemviews/kitemlistviewaccessible.cpp
index ce45ed81d01ca7462b865b8933b3e54c77843aba..0188408a7241b3e5fa0c6ca7a2268f320ed70ca7 100644 (file)
--- a/src/kitemviews/kitemlistviewaccessible.cpp
+++ b/src/kitemviews/kitemlistviewaccessible.cpp
@@ -27,8 +27,8 @@
 #include "kitemlistview.h"
 #include "private/kitemlistviewlayouter.h"
 
 #include "kitemlistview.h"
 #include "private/kitemlistviewlayouter.h"
 

-#include <qgraphicsscene.h>
-#include <qgraphicsview.h>
+#include <QGraphicsScene>
+#include <QGraphicsView>

 
 KItemListView* KItemListViewAccessible::view() const
 {
 
 KItemListView* KItemListViewAccessible::view() const
 {


@@ -44,10 +44,9 @@ KItemListViewAccessible::KItemListViewAccessible(KItemListView* view_) :
 
 KItemListViewAccessible::~KItemListViewAccessible()
 {
 
 KItemListViewAccessible::~KItemListViewAccessible()
 {

-    foreach (QAccessibleInterface* child, m_cells) {
-        if (child) {
-            QAccessible::Id childId = QAccessible::uniqueId(child);
-            QAccessible::deleteAccessibleInterface(childId);
+    foreach (AccessibleIdWrapper idWrapper, m_cells) {
+        if (idWrapper.isValid) {
+            QAccessible::deleteAccessibleInterface(idWrapper.id);

         }
     }
 }
         }
     }
 }


@@ -75,12 +74,13 @@ QAccessibleInterface* KItemListViewAccessible::cell(int index) const
     }
     Q_ASSERT(index < m_cells.size());
 
     }
     Q_ASSERT(index < m_cells.size());
 

-    QAccessibleInterface* child = m_cells.at(index);
-    if (!child) {
-        child = new KItemListAccessibleCell(view(), index);
-        QAccessible::registerAccessibleInterface(child);
+    AccessibleIdWrapper idWrapper = m_cells.at(index);
+    if (!idWrapper.isValid) {
+        idWrapper.id = QAccessible::registerAccessibleInterface(new KItemListAccessibleCell(view(), index));
+        idWrapper.isValid = true;
+        m_cells.insert(index, idWrapper);

     }
     }

-    return child;
+    return QAccessible::accessibleInterface(idWrapper.id);

 }
 
 QAccessibleInterface* KItemListViewAccessible::cellAt(int row, int column) const
 }
 
 QAccessibleInterface* KItemListViewAccessible::cellAt(int row, int column) const


@@ -265,6 +265,12 @@ QAccessibleInterface* KItemListViewAccessible::child(int index) const
     return nullptr;
 }
 
     return nullptr;
 }
 

+KItemListViewAccessible::AccessibleIdWrapper::AccessibleIdWrapper() :
+    isValid(false),
+    id(0)
+{
+}
+

 // Table Cell
 
 KItemListAccessibleCell::KItemListAccessibleCell(KItemListView* view, int index) :
 // Table Cell
 
 KItemListAccessibleCell::KItemListAccessibleCell(KItemListView* view, int index) :


@@ -415,7 +421,7 @@ int KItemListAccessibleCell::childCount() const
 
 int KItemListAccessibleCell::indexOfChild(const QAccessibleInterface* child) const
 {
 
 int KItemListAccessibleCell::indexOfChild(const QAccessibleInterface* child) const
 {

-    Q_UNUSED(child);
+    Q_UNUSED(child)

     return -1;
 }
 
     return -1;
 }