show some level cover pictures

and it looks terrible... the data is comming thru, but its flashing or freezing the window in like 0.42 secunds also...

Author: noisecode3

src/Controller.cpp

@@ -11,7 +11,7 @@
  * GNU General Public License for more details.
  */
 
-#include "Controller.hpp"
+#include "../src/Controller.hpp"
 #include <QDebug>
 
 Controller::Controller() : controllerThread(new QThread()) {
@@ -99,6 +99,10 @@ void Controller::getList(QVector<ListItemData>* list) {
     model.getList(list);
 }
 
+void Controller::getCoverList(QVector<ListItemData*>* list) {
+    model.getCoverList(list);
+}
+
 const InfoData Controller::getInfo(int id) {
     return model.getInfo(id);
 }

src/Controller.hpp

@@ -15,7 +15,7 @@
 #define SRC_CONTROLLER_HPP_
 #include <QObject>
 #include <QThread>
-#include "Model.hpp"
+#include "../src/Model.hpp"
 
 /**
  * The controller activate UI thread work or light instant work on the model
@@ -38,6 +38,7 @@ class Controller : public QObject {
     void setupLevel(int id);
 
     void getList(QVector<ListItemData>* list);
+    void getCoverList(QVector<ListItemData*>* list);
     const InfoData getInfo(int id);
     const QString getWalkthrough(int id);
     bool link(int id);

src/Data.cpp

@@ -1,5 +1,5 @@
 /* TombRaiderLinuxLauncher
- * Martin Bångens Copyright (C) 2024
+ * Martin Bångens Copyright (C) 2025
  * This program is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
  * the Free Software Foundation, either version 3 of the License, or
@@ -56,7 +56,8 @@ QVector<ListItemData> Data::getListItems() {
             "JOIN Author ON AuthorList.authorID = Author.AuthorID "
             "GROUP BY Info.trleID");
     if (!status) {
-        qDebug() << "Error preparing query getListItems:" << query.lastError().text();
+        qDebug() << "Error preparing query getListItems:"
+            << query.lastError().text();
     }
 
     if (status) {
@@ -73,51 +74,50 @@ QVector<ListItemData> Data::getListItems() {
                     query.value("Info.duration").toInt()));
             }
         } else {
-            qDebug() << "Error executing query getListItems:" << query.lastError().text();
+            qDebug() << "Error executing query getListItems:"
+                << query.lastError().text();
         }
     }
 
     return items;
 }
 
-QVector<ListItemPicture> Data::getPictures(QList<qint64> trle_ids) {
-    bool status = true;
+void Data::getCoverPictures(QVector<ListItemData*>* items) {
     QSqlQuery query(db);
-    QVector<ListItemPicture> items;
 
-    foreach(const qint64 &id, trle_ids) {
-        status = query.prepare(
-            "SELECT Info.trleID, Picture.data "
-            "FROM Level "
-            "JOIN Info ON Level.infoID = Info.InfoID "
-            "JOIN Screens ON Level.LevelID = Screens.levelID "
-            "JOIN Picture ON Screens.pictureID = Picture.PictureID "
-            "WHERE Info.trleID = :id AND Screens.position = 0 ");
-        query.bindValue(":id", id);
-        if (!status) {
-            qDebug() << "Error preparing query getPictures:" << query.lastError().text();
-        }
+    bool status = query.prepare(
+        "SELECT Info.trleID, Picture.data "
+        "FROM Level "
+        "JOIN Info ON Level.infoID = Info.InfoID "
+        "JOIN Screens ON Level.LevelID = Screens.levelID "
+        "JOIN Picture ON Screens.pictureID = Picture.PictureID "
+        "WHERE Info.trleID = :id AND Screens.position = 0 ");
+    if (!status) {
+        qDebug() << "Error preparing query getPictures:"
+            << query.lastError().text();
+    }
 
-        if (status) {
+    if (status) {
+        qint64 size = items->size();
+        for (qint64 i = size-100; i < size; i++) {
+            ListItemData* level = (*items)[i];
+            query.bindValue(":id", level->m_trle_id);
             if (query.exec()) {
                 if (query.next() == true) {
-                    items.append(ListItemPicture(
-                        query.value("Info.trleID").toInt(),
-                        query.value("Picture.data").toByteArray()));
+                    level->addPicture(
+                            query.value("Picture.data").toByteArray());
                 }
             } else {
-                qDebug() << "Error executing query getPictures:" << query.lastError().text();
+                qDebug() << "Error executing query getPictures:"
+                    << query.lastError().text();
             }
         }
     }
-
-    return items;
 }
 
 InfoData Data::getInfo(const int id) {
     QSqlQuery query(db);
     bool status = false;
-    QVector<QByteArray> imageList;
     InfoData result;
 
     status = query.prepare(
@@ -131,6 +131,7 @@ InfoData Data::getInfo(const int id) {
     if (status) {
         if (query.exec() == true) {
             if (query.next() == true) {
+                QVector<QByteArray> imageList;
                 QString body = query.value("body").toString();
                 // notice that we jump over the fist image
                 // the first image is the cover image

src/Data.hpp

@@ -92,35 +92,47 @@ struct ZipData {
     QString release;
 };
 
+
 /**
- * @struct ListItemPicture
- * @brief Represents a Tomb Raider Level Entry Card Picture.
+ * @struct ListItemData
+ * @brief Represents a Tomb Raider Level Entry Card Info.
  *
- * This struct is designed to store a single TRLE (Tomb Raider Level Editor) level picture record.
- * Each record is a cover image displayed as a card in the application.
+ * This struct is designed to store a single TRLE (Tomb Raider Level Editor) level record.
+ * Each record contains metadata and a cover image displayed as a card in the application.
+ * The struct includes properties to facilitate searching, filtering, and sorting.
  */
-struct ListItemPicture {
+struct ListItemData {
     /**
-     * @brief Default constructor for `ListItemPicture`.
+     * @brief Default constructor for `ListItemData`.
      *
-     * Initializes an empty instance of `ListItemPicture`.
+     * Initializes an empty instance of `ListItemData`.
      */
-    ListItemPicture() {}
+    ListItemData() {}
 
     /**
-     * @brief Parameterized constructor for `ListItemPicture`.
+     * @brief Parameterized constructor for `ListItemData`.
      *
-     * This constructor initializes a `ListItemPicture` object with a cover image.
-     * The image is converted from raw `QByteArray` to a `QIcon` after scaling it to
-     * fit within 640x480 dimensions. The scaling maintains the aspect ratio and
-     * smooths out pixels using `Qt::SmoothTransformation`. The image is centered
-     * within a transparent background if its aspect ratio does not perfectly match the target.
+     * This constructor initializes a `ListItemData` object with metadata.
      *
      * @param id The TRLE numeric level ID.
-     * @param imageData The cover image as a `QByteArray`. Supported formats include JPEG, PNG, and WEBP.
+     * @param title The TRLE title. Expected to contain a single name.
+     * @param author The TRLE author(s). Can be a single name or multiple names.
+     * @param type The TRLE type, represented by a numeric ID.
+     * @param classInput The TRLE class, represented by a numeric ID.
+     * @param releaseDate The release date in the format "YYYY-MM-DD" (e.g., "2000-01-01").
+     * @param difficulty The TRLE difficulty, represented by a numeric ID.
+     * @param duration The TRLE duration, represented by a numeric ID.
+     * @param m_cover The cover image as a `m_cover`.
      */
-    ListItemPicture(
-        qint64 id, QByteArray imageData) : m_trle_id(id) {
+    ListItemData(
+            qint64 id, const QString& title, const QStringList& authors,
+            qint64 type, qint64 classInput, const QString& releaseDate,
+            qint64 difficulty, qint64 duration) :
+            m_trle_id(id), m_title(title), m_authors(authors), m_type(type),
+            m_class(classInput), m_releaseDate(releaseDate),
+            m_difficulty(difficulty), m_duration(duration) {}
+
+    void addPicture(const QByteArray& imageData) {
         // Load the image from the byte array
         QPixmap pixmap;
         pixmap.loadFromData(imageData, "WEBP");
@@ -151,57 +163,9 @@ struct ListItemPicture {
         painter.drawPixmap(xOffset, yOffset, scaledPixmap);
         painter.end();
 
-        // Store the resulting pixmap in a QIcon
-        m_picture.addPixmap(centeredPixmap);
+        // Store the resulting pixmap in m_picture
+        m_cover = centeredPixmap;
     }
-
-    // Data members
-    qint64 m_trle_id;        ///< The TRLE level id.
-    QIcon m_picture;         ///< The cover image.
-};
-
-/**
- * @struct ListItemData
- * @brief Represents a Tomb Raider Level Entry Card Info.
- *
- * This struct is designed to store a single TRLE (Tomb Raider Level Editor) level record.
- * Each record contains metadata and a cover image displayed as a card in the application.
- * The struct includes properties to facilitate searching, filtering, and sorting without
- * the cover image, just a pointer.
- */
-struct ListItemData {
-    /**
-     * @brief Default constructor for `ListItemData`.
-     *
-     * Initializes an empty instance of `ListItemData`.
-     */
-    ListItemData() {}
-
-    /**
-     * @brief Parameterized constructor for `ListItemData`.
-     *
-     * This constructor initializes a `ListItemData` object with metadata.
-     *
-     * @param id The TRLE numeric level ID.
-     * @param title The TRLE title. Expected to contain a single name.
-     * @param author The TRLE author(s). Can be a single name or multiple names.
-     * @param type The TRLE type, represented by a numeric ID.
-     * @param classInput The TRLE class, represented by a numeric ID.
-     * @param releaseDate The release date in the format "YYYY-MM-DD" (e.g., "2000-01-01").
-     * @param difficulty The TRLE difficulty, represented by a numeric ID.
-     * @param duration The TRLE duration, represented by a numeric ID.
-     * @param m_cover The cover image as a `ListItemPicture*`.
-     */
-    ListItemData(
-        qint64 id, const QString& title, const QStringList& authors,
-        qint64 type, qint64 classInput, const QString& releaseDate,
-        qint64 difficulty, qint64 duration) :
-        m_trle_id(id), m_title(title), m_authors(authors), m_type(type),
-        m_class(classInput), m_releaseDate(releaseDate),
-        m_difficulty(difficulty), m_duration(duration) {
-            m_cover = nullptr;
-        }
-
     // Data members
     qint64 m_trle_id;        ///< The TRLE level id.
     QString m_title;         ///< The TRLE level title.
@@ -211,7 +175,7 @@ struct ListItemData {
     QString m_releaseDate;   ///< The release date in "YYYY-MM-DD" format.
     qint64 m_difficulty;     ///< ID of the difficulty of the level.
     qint64 m_duration;       ///< ID of the estimated duration of the level.
-    ListItemPicture* m_cover;   ///< The TRLE cover image pointer.
+    QPixmap m_cover;         ///< The TRLE cover image pointer.
 };
 
 /**
@@ -243,19 +207,18 @@ struct InfoData {
         : m_body(body) {
         for (const QByteArray& image : imageList) {
             QPixmap pixmap;
-            QIcon finalIcon;
 
             // Load image data into a QPixmap and convert it to a QIcon
-            if (pixmap.loadFromData(image, "WEBP") == true) {
-                finalIcon.addPixmap(pixmap);
+            if (!pixmap.loadFromData(image, "WEBP")) {
+                qDebug() << "Could not load webp data to QPixmap.";
             }
 
-            m_imageList.push_back(finalIcon);
+            m_imageList.push_back(pixmap);
         }
     }
 
     QString m_body;  ///< The textual content associated with this object.
-    QVector<QIcon> m_imageList;  ///< A list of icons generated from image data.
+    QVector<QPixmap> m_imageList;  ///< A list of level large screen image data.
 };
 
 class Data : public QObject {
@@ -298,7 +261,7 @@ class Data : public QObject {
 
     qint64 getListRowCount();
     QVector<ListItemData> getListItems();
-    QVector<ListItemPicture> getPictures(QList<qint64> trle_ids);
+    void getCoverPictures(QVector<ListItemData*>* items);
     InfoData getInfo(int id);
     QString getWalkthrough(int id);
     int getType(int id);

src/Model.cpp

@@ -103,6 +103,10 @@ void Model::getList(QVector<ListItemData>* list) {
     *list = data.getListItems();
 }
 
+void Model::getCoverList(QVector<ListItemData*>* list) {
+    data.getCoverPictures(list);
+}
+
 int Model::getItemState(int id) {
     int status = 0;
     if (id < 0) {

src/Model.hpp

@@ -20,11 +20,13 @@
 #include <QBitArray>
 #include <QDebug>
 #include <QtCore>
+
 #include <cassert>
-#include "Data.hpp"
-#include "FileManager.hpp"
-#include "Network.hpp"
-#include "Runner.hpp"
+
+#include "../src/Data.hpp"
+#include "../src/FileManager.hpp"
+#include "../src/Network.hpp"
+#include "../src/Runner.hpp"
 
 class InstructionManager : public QObject {
     Q_OBJECT
@@ -62,6 +64,7 @@ class Model : public QObject {
     int checkGameDirectory(int id);
     int checkLevelDirectory(int id);
     void getList(QVector<ListItemData>* list);
+    void getCoverList(QVector<ListItemData*>* list);
     int getItemState(int id);
     bool runWine(const int id);
     bool setLink(int id);

src/TombRaiderLinuxLauncher.cpp

@@ -98,8 +98,6 @@ TombRaiderLinuxLauncher::TombRaiderLinuxLauncher(QWidget *parent)
 
     connect(ui->radioButtonLevelName, &QRadioButton::clicked,
             this, &TombRaiderLinuxLauncher::sortByTitle);
-    connect(ui->radioButtonAuthor, &QRadioButton::clicked,
-            this, &TombRaiderLinuxLauncher::sortByAuthor);
     connect(ui->radioButtonDifficulty, &QRadioButton::clicked,
             this, &TombRaiderLinuxLauncher::sortByDifficulty);
     connect(ui->radioButtonDuration, &QRadioButton::clicked,
@@ -150,10 +148,6 @@ void TombRaiderLinuxLauncher::sortByTitle() {
     model->sortItems(model->compareTitles);
 }
 
-void TombRaiderLinuxLauncher::sortByAuthor() {
-    // sortItems(compareAuthors);
-}
-
 void TombRaiderLinuxLauncher::sortByDifficulty() {
     model->sortItems(model->compareDifficulties);
 }

src/TombRaiderLinuxLauncher.hpp

@@ -89,10 +89,9 @@ class TombRaiderLinuxLauncher : public QMainWindow {
      */
     void generateList(const QList<int>& availableGames);
     /**
-     * Sorts the list by author.
+     * Load more levels when scrolled to the end of the list.
      */
     void loadMoreLevels(int value);
-    void sortByAuthor();
     /**
      * Sorts the list by title.
      */