]> cloud.milkyroute.net Git - dolphin.git/blobdiff - src/kitemviews/kitemlistviewaccessible.cpp
projects / dolphin.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | inline | side by side
Merge branch 'master' into kf6
[dolphin.git] / src / kitemviews / kitemlistviewaccessible.cpp
diff --git a/src/kitemviews/kitemlistviewaccessible.cpp b/src/kitemviews/kitemlistviewaccessible.cpp
index d279879576f96493dd681d39dda020e0a70da50f..a8d80ab52baa039b63533f28b83ba62e35cc9610 100644 (file)
--- a/src/kitemviews/kitemlistviewaccessible.cpp
+++ b/src/kitemviews/kitemlistviewaccessible.cpp
@@ -1,100 +1,118 @@
+/*
+ * SPDX-FileCopyrightText: 2012 Amandeep Singh <aman.dedman@gmail.com>
+ *
+ * SPDX-License-Identifier: GPL-2.0-or-later
+ */
+
+#ifndef QT_NO_ACCESSIBILITY
 #include "kitemlistviewaccessible.h"
 #include "kitemlistviewaccessible.h"
+
+#include "kitemlistcontainer.h"
 #include "kitemlistcontroller.h"
 #include "kitemlistselectionmanager.h"
 #include "kitemlistcontroller.h"
 #include "kitemlistselectionmanager.h"
+#include "kitemlistview.h"
 #include "private/kitemlistviewlayouter.h"
 
 #include "private/kitemlistviewlayouter.h"
 
-#include <QtGui/qtableview.h>
-#include <QtGui/qaccessible2.h>
-#include <qgraphicsscene.h>
-#include <qgraphicsview.h>
-
-#include <KDebug>
-#include <QHash>
-
-#ifndef QT_NO_ACCESSIBILITY
-
-#ifndef QT_NO_ITEMVIEWS
+#include <QGraphicsScene>
+#include <QGraphicsView>
 
 KItemListView *KItemListViewAccessible::view() const
 {
 
 KItemListView *KItemListViewAccessible::view() const
 {
-    return qobject_cast<KItemListView*>(object());
+    return qobject_cast<KItemListView *>(object());
 }
 
 }
 
-KItemListViewAccessible::KItemListViewAccessible(KItemListView *view_)
-    : QAccessibleObjectEx(view_)
+KItemListViewAccessible::KItemListViewAccessible(KItemListView *view_, KItemListContainerAccessible *parent)
+    : QAccessibleObject(view_)
+    , m_parent(parent)
 {
     Q_ASSERT(view());
 {
     Q_ASSERT(view());
-
-    /*if (qobject_cast<const QTableView*>(view())) {
-        m_role = QAccessible::Table;
-    } else if (qobject_cast<const QTreeView*>(view())) {
-        m_role = QAccessible::Tree;
-    } else if (qobject_cast<const QListView*>(view())) {
-        m_role = QAccessible::List;
-    } else {
-        // is this our best guess?
-        m_role = QAccessible::Table;
-    }*/
+    Q_CHECK_PTR(parent);
+    m_cells.resize(childCount());
 }
 
 KItemListViewAccessible::~KItemListViewAccessible()
 {
 }
 
 KItemListViewAccessible::~KItemListViewAccessible()
 {
+    for (AccessibleIdWrapper idWrapper : std::as_const(m_cells)) {
+        if (idWrapper.isValid) {
+            QAccessible::deleteAccessibleInterface(idWrapper.id);
+        }
+    }
 }
 
 }
 
-void KItemListViewAccessible::modelReset()
-{}
+void *KItemListViewAccessible::interface_cast(QAccessible::InterfaceType type)
+{
+    if (type == QAccessible::TableInterface) {
+        return static_cast<QAccessibleTableInterface *>(this);
+    }
+    return nullptr;
+}
 
 
-QAccessibleTable2CellInterface *KItemListViewAccessible::cell(int index) const
+void KItemListViewAccessible::modelReset()
 {
 {
-    if (index > 0)
-        return new KItemListAccessibleCell(view(), index);
-    return 0;
 }
 
 }
 
-QAccessibleTable2CellInterface *KItemListViewAccessible::cellAt(int row, int column) const
+QAccessibleInterface *KItemListViewAccessible::cell(int index) const
 {
 {
-    /*Q_ASSERT(role(0) != QAccessible::Tree);
-    QModelIndex index = view()->model()->index(row, column);
-    //Q_ASSERT(index.isValid());
-    if (!index.isValid()) {
-        qWarning() << "QAccessibleTable2::cellAt: invalid index: " << index << " for " << view();
-        return 0;
+    if (index < 0 || index >= view()->model()->count()) {
+        return nullptr;
+    }
+
+    if (m_cells.size() <= index) {
+        m_cells.resize(childCount());
+    }
+    Q_ASSERT(index < m_cells.size());
+
+    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 cell(index);*/
-    Q_UNUSED(row)
-    Q_UNUSED(column)
-    return cell(1);
+    return QAccessible::accessibleInterface(idWrapper.id);
+}
 
 
+QAccessibleInterface *KItemListViewAccessible::cellAt(int row, int column) const
+{
+    return cell(columnCount() * row + column);
 }
 
 QAccessibleInterface *KItemListViewAccessible::caption() const
 {
 }
 
 QAccessibleInterface *KItemListViewAccessible::caption() const
 {
-    return 0;
+    return nullptr;
 }
 
 QString KItemListViewAccessible::columnDescription(int) const
 {
 }
 
 QString KItemListViewAccessible::columnDescription(int) const
 {
-    // FIXME: no i18n
-    return QObject::tr("No Column Description");
+    return QString();
 }
 
 int KItemListViewAccessible::columnCount() const
 {
 }
 
 int KItemListViewAccessible::columnCount() const
 {
-    return view()->layouter()->columnCount();
+    return view()->m_layouter->columnCount();
 }
 
 int KItemListViewAccessible::rowCount() const
 {
 }
 
 int KItemListViewAccessible::rowCount() const
 {
+    if (columnCount() <= 0) {
+        return 0;
+    }
+
     int itemCount = view()->model()->count();
     int rowCount = itemCount / columnCount();
     int itemCount = view()->model()->count();
     int rowCount = itemCount / columnCount();
-    if (itemCount % rowCount)
+
+    if (rowCount <= 0) {
+        return 0;
+    }
+
+    if (itemCount % columnCount()) {
         ++rowCount;
         ++rowCount;
+    }
     return rowCount;
 }
 
 int KItemListViewAccessible::selectedCellCount() const
 {
     return rowCount;
 }
 
 int KItemListViewAccessible::selectedCellCount() const
 {
-    return view()->controller()->selectionManager()->selectedItems().size();
+    return view()->controller()->selectionManager()->selectedItems().count();
 }
 
 int KItemListViewAccessible::selectedColumnCount() const
 }
 
 int KItemListViewAccessible::selectedColumnCount() const
@@ -109,13 +127,15 @@ int KItemListViewAccessible::selectedRowCount() const
 
 QString KItemListViewAccessible::rowDescription(int) const
 {
 
 QString KItemListViewAccessible::rowDescription(int) const
 {
-    return "No Row Description";
+    return QString();
 }
 
 }
 
-QList<QAccessibleTable2CellInterface*> KItemListViewAccessible::selectedCells() const
+QList<QAccessibleInterface *> KItemListViewAccessible::selectedCells() const
 {
 {
-    QList<QAccessibleTable2CellInterface*> cells;
-    Q_FOREACH (int index, view()->controller()->selectionManager()->selectedItems()) {
+    QList<QAccessibleInterface *> cells;
+    const auto items = view()->controller()->selectionManager()->selectedItems();
+    cells.reserve(items.count());
+    for (int index : items) {
         cells.append(cell(index));
     }
     return cells;
         cells.append(cell(index));
     }
     return cells;
@@ -123,202 +143,137 @@ QList<QAccessibleTable2CellInterface*> KItemListViewAccessible::selectedCells()
 
 QList<int> KItemListViewAccessible::selectedColumns() const
 {
 
 QList<int> KItemListViewAccessible::selectedColumns() const
 {
-    QList<int> columns;
-    /*Q_FOREACH (const QModelIndex &index, view()->selectionModel()->selectedColumns()) {
-        columns.append(index.column());
-    }*/
-    return columns;
+    return QList<int>();
 }
 
 QList<int> KItemListViewAccessible::selectedRows() const
 {
 }
 
 QList<int> KItemListViewAccessible::selectedRows() const
 {
-    QList<int> rows;
-    /*Q_FOREACH (const QModelIndex &index, view()->selectionModel()->selectedRows()) {
-        rows.append(index.row());
-    }*/
-    return rows;
+    return QList<int>();
 }
 
 QAccessibleInterface *KItemListViewAccessible::summary() const
 {
 }
 
 QAccessibleInterface *KItemListViewAccessible::summary() const
 {
-    return 0;
+    return nullptr;
 }
 
 bool KItemListViewAccessible::isColumnSelected(int) const
 {
 }
 
 bool KItemListViewAccessible::isColumnSelected(int) const
 {
-    return false; //view()->selectionModel()->isColumnSelected(column, QModelIndex());
+    return false;
 }
 
 bool KItemListViewAccessible::isRowSelected(int) const
 {
 }
 
 bool KItemListViewAccessible::isRowSelected(int) const
 {
-    return false; //view()->selectionModel()->isRowSelected(row, QModelIndex());
+    return false;
 }
 
 bool KItemListViewAccessible::selectRow(int)
 {
 }
 
 bool KItemListViewAccessible::selectRow(int)
 {
-    /*QModelIndex index = view()->model()->index(row, 0);
-    if (!index.isValid() || view()->selectionMode() & QAbstractItemView::NoSelection)
-        return false;
-    view()->selectionModel()->select(index, QItemSelectionModel::Select);*/
     return true;
 }
 
 bool KItemListViewAccessible::selectColumn(int)
 {
     return true;
 }
 
 bool KItemListViewAccessible::selectColumn(int)
 {
-    /*QModelIndex index = view()->model()->index(0, column);
-    if (!index.isValid() || view()->selectionMode() & QAbstractItemView::NoSelection)
-        return false;
-    view()->selectionModel()->select(index, QItemSelectionModel::Select);*/
     return true;
 }
 
 bool KItemListViewAccessible::unselectRow(int)
 {
     return true;
 }
 
 bool KItemListViewAccessible::unselectRow(int)
 {
-    /*QModelIndex index = view()->model()->index(row, 0);
-    if (!index.isValid() || view()->selectionMode() & QAbstractItemView::NoSelection)
-        return false;
-    view()->selectionModel()->select(index, QItemSelectionModel::Deselect);*/
     return true;
 }
 
 bool KItemListViewAccessible::unselectColumn(int)
 {
     return true;
 }
 
 bool KItemListViewAccessible::unselectColumn(int)
 {
-    /*QModelIndex index = view()->model()->index(0, column);
-    if (!index.isValid() || view()->selectionMode() & QAbstractItemView::NoSelection)
-        return false;
-    view()->selectionModel()->select(index, QItemSelectionModel::Columns & QItemSelectionModel::Deselect);*/
     return true;
 }
 
     return true;
 }
 
-QAccessible2::TableModelChange KItemListViewAccessible::modelChange() const
+void KItemListViewAccessible::modelChange(QAccessibleTableModelChangeEvent * /*event*/)
 {
 {
-    QAccessible2::TableModelChange change;
-    // FIXME
-    return change;
 }
 
 }
 
-QAccessible::Role KItemListViewAccessible::role(int child) const
+QAccessible::Role KItemListViewAccessible::role() const
 {
 {
-    Q_ASSERT(child >= 0);
-    if (child > 0)
-        return QAccessible::Cell;
     return QAccessible::Table;
 }
 
     return QAccessible::Table;
 }
 
-QAccessible::State KItemListViewAccessible::state(int child) const
+QAccessible::State KItemListViewAccessible::state() const
 {
 {
-    Q_ASSERT(child == 0);
-    return QAccessible::Normal;
+    QAccessible::State s;
+    return s;
 }
 
 }
 
-int KItemListViewAccessible::childAt(int x, int y) const
+QAccessibleInterface *KItemListViewAccessible::childAt(int x, int y) const
 {
 {
-    QPointF point = QPointF(x,y);
-    return view()->itemAt(view()->mapFromScene(point));
+    const QPointF point = QPointF(x, y);
+    const std::optional<int> itemIndex = view()->itemAt(view()->mapFromScene(point));
+    return child(itemIndex.value_or(-1));
 }
 
 }
 
-int KItemListViewAccessible::childCount() const
+QAccessibleInterface *KItemListViewAccessible::parent() const
 {
 {
-    return rowCount() * columnCount();
+    return m_parent;
 }
 
 }
 
-int KItemListViewAccessible::indexOfChild(const QAccessibleInterface *iface) const
+int KItemListViewAccessible::childCount() const
 {
 {
-    /*Q_ASSERT(iface->role(0) != QAccessible::TreeItem); // should be handled by tree class
-    if (iface->role(0) == QAccessible::Cell || iface->role(0) == QAccessible::ListItem) {
-        const QAccessibleTable2Cell* cell = static_cast<const QAccessibleTable2Cell*>(iface);
-        return logicalIndex(cell->m_index);
-    } else if (iface->role(0) == QAccessible::ColumnHeader){
-        const QAccessibleTable2HeaderCell* cell = static_cast<const QAccessibleTable2HeaderCell*>(iface);
-        return cell->index + (verticalHeader() ? 1 : 0) + 1;
-    } else if (iface->role(0) == QAccessible::RowHeader){
-        const QAccessibleTable2HeaderCell* cell = static_cast<const QAccessibleTable2HeaderCell*>(iface);
-        return (cell->index+1) * (view()->model()->rowCount()+1)  + 1;
-    } else if (iface->role(0) == QAccessible::Pane) {
-        return 1; // corner button
-    } else {
-        qWarning() << "WARNING QAccessibleTable2::indexOfChild Fix my children..."
-                   << iface->role(0) << iface->text(QAccessible::Name, 0);
-    }
-    // FIXME: we are in denial of our children. this should stop.
-    return -1;*/
-    
-    const KItemListAccessibleCell *widget = static_cast<const KItemListAccessibleCell*>(iface);
-    return widget->getIndex();
+    return view()->model()->count();
 }
 
 }
 
-QString KItemListViewAccessible::text(Text t, int child) const
+int KItemListViewAccessible::indexOfChild(const QAccessibleInterface *interface) const
 {
 {
-    Q_ASSERT(child == 0);
-    // FIXME: I don't think this is needed, but if at all it needs i18n
-    if (t == QAccessible::Description)
-        return QObject::tr("List of files present in the current directory");
-    return QObject::tr("File List");
+    const KItemListAccessibleCell *widget = static_cast<const KItemListAccessibleCell *>(interface);
+    return widget->index();
 }
 
 }
 
-QRect KItemListViewAccessible::rect(int child) const
+QString KItemListViewAccessible::text(QAccessible::Text) const
 {
 {
-    Q_UNUSED(child)
-    if (!view()->isVisible())
-        return QRect();
-    QPoint origin = view()->scene()->views()[0]->mapToGlobal(QPoint(0, 0));
-    QRect viewRect = view()->geometry().toRect();
-    return viewRect.translated(origin);
+    return QString();
 }
 
 }
 
-int KItemListViewAccessible::navigate(RelationFlag relation, int index, QAccessibleInterface **iface) const
+QRect KItemListViewAccessible::rect() const
 {
 {
-    *iface = 0;
-    switch (relation) {
-    /*case Ancestor: {
-        if (index == 1 && view()->parent()) {
-            *iface = QAccessible::queryAccessibleInterface(view()->parent());
-            if (*iface)
-                return 0;
-        }
-        break;
-    }*/
-    case QAccessible::Child: {
-        Q_ASSERT(index > 0);
-        *iface = cell(index);
-        if (*iface) {
-            return 0;
-        }
-        break;
+    if (!view()->isVisible()) {
+        return QRect();
     }
     }
-    default:
-        break;
+
+    const QGraphicsScene *scene = view()->scene();
+    if (scene) {
+        const QPoint origin = scene->views().at(0)->mapToGlobal(QPoint(0, 0));
+        const QRect viewRect = view()->geometry().toRect();
+        return viewRect.translated(origin);
+    } else {
+        return QRect();
     }
     }
-    return -1;
 }
 
 }
 
-QAccessible::Relation KItemListViewAccessible::relationTo(int, const QAccessibleInterface *, int) const
+QAccessibleInterface *KItemListViewAccessible::child(int index) const
 {
 {
-    return QAccessible::Unrelated;
+    if (index >= 0 && index < childCount()) {
+        return cell(index);
+    }
+    return nullptr;
 }
 
 }
 
-#ifndef QT_NO_ACTION
-int KItemListViewAccessible::userActionCount(int) const
+KItemListViewAccessible::AccessibleIdWrapper::AccessibleIdWrapper()
+    : isValid(false)
+    , id(0)
 {
 {
-    return 0;
-}
-QString KItemListViewAccessible::actionText(int, Text, int) const
-{
-    return QString();
 }
 }
-bool KItemListViewAccessible::doAction(int, int, const QVariantList &)
+
+// Table Cell
+
+KItemListAccessibleCell::KItemListAccessibleCell(KItemListView *view, int index)
+    : m_view(view)
+    , m_index(index)
 {
 {
-    return false;
+    Q_ASSERT(index >= 0 && index < view->model()->count());
 }
 }
-#endif
 
 
-// TABLE CELL
-
-KItemListAccessibleCell::KItemListAccessibleCell(KItemListView *view_, int index_)
-    : /* QAccessibleSimpleEditableTextInterface(this), */ view(view_)
-    , index(index_)
+void *KItemListAccessibleCell::interface_cast(QAccessible::InterfaceType type)
 {
 {
-    Q_ASSERT(index_ > 0);
+    if (type == QAccessible::TableCellInterface) {
+        return static_cast<QAccessibleTableCellInterface *>(this);
+    }
+    return nullptr;
 }
 
 int KItemListAccessibleCell::columnExtent() const
 }
 
 int KItemListAccessibleCell::columnExtent() const
@@ -331,246 +286,182 @@ int KItemListAccessibleCell::rowExtent() const
     return 1;
 }
 
     return 1;
 }
 
-QList<QAccessibleInterface*> KItemListAccessibleCell::rowHeaderCells() const
+QList<QAccessibleInterface *> KItemListAccessibleCell::rowHeaderCells() const
 {
 {
-    QList<QAccessibleInterface*> headerCell;
-    /*if (verticalHeader()) {
-        headerCell.append(new QAccessibleTable2HeaderCell(view, m_index.row(), Qt::Vertical));
-    }*/
-    return headerCell;
+    return QList<QAccessibleInterface *>();
 }
 
 }
 
-QList<QAccessibleInterface*> KItemListAccessibleCell::columnHeaderCells() const
+QList<QAccessibleInterface *> KItemListAccessibleCell::columnHeaderCells() const
 {
 {
-    QList<QAccessibleInterface*> headerCell;
-    /*if (horizontalHeader()) {
-        headerCell.append(new QAccessibleTable2HeaderCell(view, m_index.column(), Qt::Horizontal));
-    }*/
-    return headerCell;
+    return QList<QAccessibleInterface *>();
 }
 
 int KItemListAccessibleCell::columnIndex() const
 {
 }
 
 int KItemListAccessibleCell::columnIndex() const
 {
-    return view->layouter()->itemColumn(index);
+    return m_view->m_layouter->itemColumn(m_index);
 }
 
 int KItemListAccessibleCell::rowIndex() const
 {
 }
 
 int KItemListAccessibleCell::rowIndex() const
 {
-    /*if (role(0) == QAccessible::TreeItem) {
-       const QTreeView *treeView = qobject_cast<const QTreeView*>(view);
-       Q_ASSERT(treeView);
-       int row = treeView->d_func()->viewIndex(m_index);
-       return row;
-    }*/
-    return view->layouter()->itemRow(index);
+    return m_view->m_layouter->itemRow(m_index);
 }
 
 }
 
-//Done
 bool KItemListAccessibleCell::isSelected() const
 {
 bool KItemListAccessibleCell::isSelected() const
 {
-    //return widget->isSelected();
-    // FIXME
-    return false;
+    return m_view->controller()->selectionManager()->isSelected(m_index);
 }
 
 }
 
-void KItemListAccessibleCell::rowColumnExtents(int *row, int *column, int *rowExtents, int *columnExtents, bool *selected) const
+QAccessibleInterface *KItemListAccessibleCell::table() const
 {
 {
-    KItemListViewLayouter* layouter = view->layouter();
-    *row = layouter->itemRow(index);
-    *column = layouter->itemColumn(index);
-    *rowExtents = 1;
-    *columnExtents = 1;
-    *selected = isSelected();
+    return QAccessible::queryAccessibleInterface(m_view);
 }
 
 }
 
-QAccessibleTable2Interface* KItemListAccessibleCell::table() const
+QAccessible::Role KItemListAccessibleCell::role() const
 {
 {
-    return QAccessible::queryAccessibleInterface(view)->table2Interface();
-}
-
-QAccessible::Role KItemListAccessibleCell::role(int child) const
-{
-    Q_ASSERT(child == 0);
     return QAccessible::Cell;
 }
 
     return QAccessible::Cell;
 }
 
-QAccessible::State KItemListAccessibleCell::state(int child) const
+QAccessible::State KItemListAccessibleCell::state() const
 {
 {
-    Q_ASSERT(child == 0);
-    QAccessible::State st = Normal;
+    QAccessible::State state;
 
 
-    //QRect globalRect = view->rect();
-    //globalRect.translate(view->mapToGlobal(QPoint(0,0)));
-    //if (!globalRect.intersects(rect(0)))
-    //    st |= Invisible;
+    state.selectable = true;
+    if (isSelected()) {
+        state.selected = true;
+    }
 
 
-//     if (widget->isSelected())
-//         st |= Selected;
-    if (view->controller()->selectionManager()->currentItem() == index)
-        st |= Focused;
+    state.focusable = true;
+    if (m_view->controller()->selectionManager()->currentItem() == m_index) {
+        state.focused = true;
+    }
 
 
-    //if (m_index.model()->data(m_index, Qt::CheckStateRole).toInt() == Qt::Checked)
-    //    st |= Checked;
-    //if (flags & Qt::ItemIsSelectable) {
-    st |= Selectable;
-    st |= Focusable;
-    if (view->controller()->selectionBehavior() == KItemListController::MultiSelection)
-        st |= MultiSelectable;
+    if (m_view->controller()->selectionBehavior() == KItemListController::MultiSelection) {
+        state.multiSelectable = true;
+    }
 
 
-        //if (view->selectionMode() == QAbstractItemView::ExtendedSelection)
-            //st |= ExtSelectable;
-    //}
-    //if (m_role == QAccessible::TreeItem) {
-    //    const QTreeView *treeView = qobject_cast<const QTreeView*>(view);
-    //    if (treeView->isExpanded(m_index))
-    //        st |= Expanded;
-    //}
+    if (m_view->model()->isExpandable(m_index)) {
+        if (m_view->model()->isExpanded(m_index)) {
+            state.expanded = true;
+        } else {
+            state.collapsed = true;
+        }
+    }
 
 
-    return st;
+    return state;
 }
 
 }
 
-//Done
 bool KItemListAccessibleCell::isExpandable() const
 {
 bool KItemListAccessibleCell::isExpandable() const
 {
-    return false; //view->model()->hasChildren(m_index);
+    return m_view->model()->isExpandable(m_index);
 }
 
 }
 
-//Done
-QRect KItemListAccessibleCell::rect(int) const
+QRect KItemListAccessibleCell::rect() const
 {
 {
-    QRect r = view->itemRect(index).toRect();
-    if (r.isNull())
+    QRect rect = m_view->itemRect(m_index).toRect();
+
+    if (rect.isNull()) {
         return QRect();
         return QRect();
-    r.translate(view->mapToScene(QPointF(0.0, 0.0)).toPoint());
-    r.translate(view->scene()->views()[0]->mapToGlobal(QPoint(0, 0)));
-    return r;
+    }
+
+    rect.translate(m_view->mapToScene(QPointF(0.0, 0.0)).toPoint());
+    rect.translate(m_view->scene()->views()[0]->mapToGlobal(QPoint(0, 0)));
+    return rect;
 }
 
 }
 
-QString KItemListAccessibleCell::text(QAccessible::Text t, int child) const
+QString KItemListAccessibleCell::text(QAccessible::Text t) const
 {
 {
-    Q_ASSERT(child == 0);
-    QHash<QByteArray, QVariant> data = view->model()->data(index);
     switch (t) {
     switch (t) {
-    case QAccessible::Value:
-    case QAccessible::Name:
+    case QAccessible::Name: {
+        const QHash<QByteArray, QVariant> data = m_view->model()->data(m_index);
         return data["text"].toString();
         return data["text"].toString();
-    case QAccessible::Description:
-        return data["text"].toString() + " : " + data["group"].toString();
+    }
+
     default:
         break;
     }
     default:
         break;
     }
+
     return QString();
 }
 
     return QString();
 }
 
-void KItemListAccessibleCell::setText(QAccessible::Text /*t*/, int child, const QString &text)
+void KItemListAccessibleCell::setText(QAccessible::Text, const QString &)
 {
 {
-    Q_ASSERT(child == 0);
-    // FIXME - is this even allowed on the KItemListWidget?
 }
 
 }
 
-bool KItemListAccessibleCell::isValid() const
+QAccessibleInterface *KItemListAccessibleCell::child(int) const
 {
 {
-    return view && (index > 0);
+    return nullptr;
 }
 
 }
 
-int KItemListAccessibleCell::navigate(RelationFlag relation, int index, QAccessibleInterface **iface) const
+bool KItemListAccessibleCell::isValid() const
 {
 {
-    if (relation == Ancestor && index == 1) {
-        //if (m_role == QAccessible::TreeItem) {
-        //    *iface = new QAccessibleTree(view);
-        //} else {
-        *iface = new KItemListViewAccessible(view);
-        return 0;
-    }
-
-    *iface = 0;
-    if (!view)
-        return -1;
-
-    switch (relation) {
-
-    case Child: {
-        return -1;
-    }
-    case Sibling:
-        if (index > 0) {
-            QAccessibleInterface *parent = queryAccessibleInterface(view);
-            int ret = parent->navigate(QAccessible::Child, index, iface);
-            delete parent;
-            if (*iface)
-                return ret;
-        }
-        return -1;
-    default:
-        break;
-    }
-
-    return -1;
+    return m_view && (m_index >= 0) && (m_index < m_view->model()->count());
 }
 
 }
 
-QAccessible::Relation KItemListAccessibleCell::relationTo(int child, const QAccessibleInterface *, int otherChild) const
+QAccessibleInterface *KItemListAccessibleCell::childAt(int, int) const
 {
 {
-    Q_ASSERT(child == 0);
-    Q_ASSERT(otherChild == 0);
-    /* we only check for parent-child relationships in trees
-    if (m_role == QAccessible::TreeItem && other->role(0) == QAccessible::TreeItem) {
-        QModelIndex otherIndex = static_cast<const QAccessibleTable2Cell*>(other)->m_index;
-        // is the other our parent?
-        if (otherIndex.parent() == m_index)
-            return QAccessible::Ancestor;
-        // are we the other's child?
-        if (m_index.parent() == otherIndex)
-            return QAccessible::Child;
-    }*/
-    return QAccessible::Unrelated;
+    return nullptr;
 }
 
 }
 
-#ifndef QT_NO_ACTION
-int KItemListAccessibleCell::userActionCount(int) const
+int KItemListAccessibleCell::childCount() const
 {
     return 0;
 }
 
 {
     return 0;
 }
 
-QString KItemListAccessibleCell::actionText(int, Text, int) const
+int KItemListAccessibleCell::indexOfChild(const QAccessibleInterface *child) const
 {
 {
-    return QString();
+    Q_UNUSED(child)
+    return -1;
 }
 
 }
 
-bool KItemListAccessibleCell::doAction(int, int, const QVariantList &)
+QAccessibleInterface *KItemListAccessibleCell::parent() const
 {
 {
-    return false;
+    return QAccessible::queryAccessibleInterface(m_view);
 }
 
 }
 
-#endif
+int KItemListAccessibleCell::index() const
+{
+    return m_index;
+}
 
 
+QObject *KItemListAccessibleCell::object() const
+{
+    return nullptr;
+}
+
+// Container Interface
 KItemListContainerAccessible::KItemListContainerAccessible(KItemListContainer *container)
 KItemListContainerAccessible::KItemListContainerAccessible(KItemListContainer *container)
-    : QAccessibleWidgetEx(container)
-{}
+    : QAccessibleWidget(container)
+{
+}
 
 
-KItemListContainerAccessible::~KItemListContainerAccessible ()
-{}
+KItemListContainerAccessible::~KItemListContainerAccessible()
+{
+}
 
 
-int KItemListContainerAccessible::childCount () const
+int KItemListContainerAccessible::childCount() const
 {
     return 1;
 }
 
 {
     return 1;
 }
 
-int KItemListContainerAccessible::indexOfChild ( const QAccessibleInterface * child ) const
+int KItemListContainerAccessible::indexOfChild(const QAccessibleInterface *child) const
 {
 {
-    if(child->object() == container()->controller()->view())
-        return 1;
+    if (child->object() == container()->controller()->view()) {
+        return 0;
+    }
     return -1;
 }
 
     return -1;
 }
 
-int KItemListContainerAccessible::navigate ( QAccessible::RelationFlag relation, int index, QAccessibleInterface ** target ) const
+QAccessibleInterface *KItemListContainerAccessible::child(int index) const
 {
 {
-    if (relation == QAccessible::Child) {
-        *target = new KItemListViewAccessible(container()->controller()->view());
-        return 0;
+    if (index == 0) {
+        return QAccessible::queryAccessibleInterface(container()->controller()->view());
     }
     }
-    return QAccessibleWidgetEx::navigate(relation, index, target);
+    return nullptr;
 }
 
 }
 
-#endif // QT_NO_ITEMVIEWS
+const KItemListContainer *KItemListContainerAccessible::container() const
+{
+    return qobject_cast<KItemListContainer *>(object());
+}
 
 #endif // QT_NO_ACCESSIBILITY
 
 #endif // QT_NO_ACCESSIBILITY