Use a QSortFilterProxyModel instead of resetting the model

CMakeLists.txt

@@ -350,7 +350,7 @@ set(SRC_FILES
 	src/MemberList.cpp
 	src/MxcImageProvider.cpp
 	src/Olm.cpp
-	src/ReadReceiptsModel.cpp
+    src/ReadReceiptsModel.cpp
 	src/RegisterPage.cpp
 	src/SSOHandler.cpp
 	src/CombinedImagePackModel.cpp
@@ -555,7 +555,7 @@ qt5_wrap_cpp(MOC_HEADERS
 	src/MainWindow.h
 	src/MemberList.h
 	src/MxcImageProvider.h
-	src/ReadReceiptsModel.h
+    src/ReadReceiptsModel.h
 	src/RegisterPage.h
 	src/SSOHandler.h
 	src/CombinedImagePackModel.h

resources/qml/ReadReceipts.qml

@@ -10,7 +10,7 @@ import im.nheko 1.0
 ApplicationWindow {
     id: readReceiptsRoot
 
-    property ReadReceiptsModel readReceipts
+    property ReadReceiptsProxy readReceipts
 
     x: MainWindow.x + (MainWindow.width / 2) - (width / 2)
     y: MainWindow.y + (MainWindow.height / 2) - (height / 2)
@@ -86,7 +86,7 @@ ApplicationWindow {
                             ToolTip.text: model.mxid
 
                             TapHandler {
-                                onSingleTapped: chat.model.openUserProfile(userId)
+                                onSingleTapped: Rooms.currentRoom.openUserProfile(userId)
                             }
 
                             CursorShape {

src/ReadReceiptsModel.cpp

@@ -46,10 +46,13 @@ ReadReceiptsModel::update()
 QHash<int, QByteArray>
 ReadReceiptsModel::roleNames() const
 {
-        return {{Mxid, "mxid"},
+        // Note: RawTimestamp is purposely not included here
-                {DisplayName, "displayName"},
+        return {
-                {AvatarUrl, "avatarUrl"},
+          {Mxid, "mxid"},
-                {Timestamp, "timestamp"}};
+          {DisplayName, "displayName"},
+          {AvatarUrl, "avatarUrl"},
+          {Timestamp, "timestamp"},
+        };
 }
 
 QVariant
@@ -67,6 +70,8 @@ ReadReceiptsModel::data(const QModelIndex &index, int role) const
                 return cache::avatarUrl(room_id_, readReceipts_[index.row()].first);
         case Timestamp:
                 return dateFormat(readReceipts_[index.row()].second);
+        case RawTimestamp:
+                return readReceipts_[index.row()].second;
         default:
                 return {};
         }
@@ -76,21 +81,22 @@ void
 ReadReceiptsModel::addUsers(
   const std::multimap<uint64_t, std::string, std::greater<uint64_t>> &users)
 {
-        beginResetModel();
+        auto newReceipts = users.size() - readReceipts_.size();
 
-        readReceipts_.clear();
+        if (newReceipts > 0) {
-        for (const auto &user : users) {
+                beginInsertRows(
-                readReceipts_.push_back({QString::fromStdString(user.second),
+                  QModelIndex{}, readReceipts_.size(), readReceipts_.size() + newReceipts - 1);
-                                         QDateTime::fromMSecsSinceEpoch(user.first)});
-        }
 
-        std::sort(readReceipts_.begin(),
+                for (const auto &user : users) {
-                  readReceipts_.end(),
+                        QPair<QString, QDateTime> item = {
-                  [](const QPair<QString, QDateTime> &a, const QPair<QString, QDateTime> &b) {
+                          QString::fromStdString(user.second),
-                          return a.second > b.second;
+                          QDateTime::fromMSecsSinceEpoch(user.first)};
-                  });
+                        if (!readReceipts_.contains(item))
+                                readReceipts_.push_back(item);
+                }
 
-        endResetModel();
+                endInsertRows();
+        }
 }
 
 QString
@@ -112,3 +118,18 @@ ReadReceiptsModel::dateFormat(const QDateTime &then) const
 
         return QLocale::system().toString(then.time(), QLocale::ShortFormat);
 }
+
+ReadReceiptsProxy::ReadReceiptsProxy(QString event_id, QString room_id, QObject *parent)
+  : QSortFilterProxyModel{parent}
+  , model_{event_id, room_id, this}
+{
+        setSourceModel(&model_);
+        setSortRole(ReadReceiptsModel::RawTimestamp);
+}
+
+bool
+ReadReceiptsProxy::lessThan(const QModelIndex &source_left, const QModelIndex &source_right) const
+{
+        // since we are sorting from greatest to least timestamp, return something that looks totally backwards!
+        return source_left.data().toULongLong() > source_right.data().toULongLong();
+}

src/ReadReceiptsModel.h

@@ -8,15 +8,13 @@
 #include <QAbstractListModel>
 #include <QDateTime>
 #include <QObject>
+#include <QSortFilterProxyModel>
 #include <QString>
 
 class ReadReceiptsModel : public QAbstractListModel
 {
         Q_OBJECT
 
-        Q_PROPERTY(QString eventId READ eventId CONSTANT)
-        Q_PROPERTY(QString roomId READ roomId CONSTANT)
-
 public:
         enum Roles
         {
@@ -24,6 +22,7 @@ public:
                 DisplayName,
                 AvatarUrl,
                 Timestamp,
+                RawTimestamp,
         };
 
         explicit ReadReceiptsModel(QString event_id, QString room_id, QObject *parent = nullptr);
@@ -51,4 +50,26 @@ private:
         QVector<QPair<QString, QDateTime>> readReceipts_;
 };
 
+class ReadReceiptsProxy : public QSortFilterProxyModel
+{
+        Q_OBJECT
+
+        Q_PROPERTY(QString eventId READ eventId CONSTANT)
+        Q_PROPERTY(QString roomId READ roomId CONSTANT)
+
+public:
+        explicit ReadReceiptsProxy(QString event_id, QString room_id, QObject *parent = nullptr);
+
+        QString eventId() const { return event_id_; }
+        QString roomId() const { return room_id_; }
+
+        bool lessThan(const QModelIndex &source_left, const QModelIndex &source_right) const;
+
+private:
+        QString event_id_;
+        QString room_id_;
+
+        ReadReceiptsModel model_;
+};
+
 #endif // READRECEIPTSMODEL_H

src/timeline/TimelineModel.cpp

@@ -1092,7 +1092,7 @@ TimelineModel::relatedInfo(QString id)
 void
 TimelineModel::showReadReceipts(QString id)
 {
-        emit openReadReceiptsDialog(new ReadReceiptsModel{id, roomId(), this});
+        emit openReadReceiptsDialog(new ReadReceiptsProxy{id, roomId(), this});
 }
 
 void

src/timeline/TimelineModel.h

@@ -349,7 +349,7 @@ signals:
         void typingUsersChanged(std::vector<QString> users);
         void replyChanged(QString reply);
         void editChanged(QString reply);
-        void openReadReceiptsDialog(ReadReceiptsModel *rr);
+        void openReadReceiptsDialog(ReadReceiptsProxy *rr);
         void paginationInProgressChanged(const bool);
         void newCallEvent(const mtx::events::collections::TimelineEvents &event);
         void scrollToIndex(int index);

src/timeline/TimelineViewManager.cpp

@@ -206,12 +206,12 @@ TimelineViewManager::TimelineViewManager(CallManager *callManager, ChatPage *par
           0,
           "InviteesModel",
           "InviteesModel needs to be instantiated on the C++ side");
-        qmlRegisterUncreatableType<ReadReceiptsModel>(
+        qmlRegisterUncreatableType<ReadReceiptsProxy>(
           "im.nheko",
           1,
           0,
-          "ReadReceiptsModel",
+          "ReadReceiptsProxy",
-          "ReadReceiptsModel needs to be instantiated on the C++ side");
+          "ReadReceiptsProxy needs to be instantiated on the C++ side");
 
         static auto self = this;
         qmlRegisterSingletonType<MainWindow>(