TreeModel: Introduce a StaticTreeItem

This splits out the case of static string displays from the
TreeItem base class, making the base more lightweight.

Change-Id: If1f442011ec60094399a41b65d9b5015f432f82e
Reviewed-by: Eike Ziller <eike.ziller@qt.io>

1 files changed, 7 insertions, 3 deletions

diff --git a/src/plugins/coreplugin/locator/locatorsettingspage.cpp b/src/plugins/coreplugin/locator/locatorsettingspage.cpp
index 5cdec51524..bfb1a001bb 100644
--- a/src/plugins/coreplugin/locator/locatorsettingspage.cpp
+++ b/src/plugins/coreplugin/locator/locatorsettingspage.cpp
@@ -72,8 +72,10 @@ class CategoryItem : public TreeItem
 public:
     CategoryItem(const QString &name, int order);
     QVariant data(int column, int role) const override;
+    Qt::ItemFlags flags(int column) const override { Q_UNUSED(column); return Qt::ItemIsEnabled; }
 
 private:
+    QString m_name;
     int m_order;
 };
 
@@ -146,16 +148,18 @@ ILocatorFilter *FilterItem::filter() const
 }
 
 CategoryItem::CategoryItem(const QString &name, int order)
-    : TreeItem(QStringList(name)),
-      m_order(order)
+    : m_name(name), m_order(order)
 {
 }
 
 QVariant CategoryItem::data(int column, int role) const
 {
+    Q_UNUSED(column);
     if (role == SortRole)
         return m_order;
-    return TreeItem::data(column, role);
+    if (role == Qt::DisplayRole)
+        return m_name;
+    return QVariant();
 }
 
 LocatorSettingsPage::LocatorSettingsPage(Locator *plugin)