aspectRation handling for images, other then images types of downloads, progress fix, labels, context menu to open files

ui/widgets/conversation.cpp

@@ -295,21 +295,6 @@ void Conversation::responseLocalFile(const QString& messageId, const QString& pa
     line->responseLocalFile(messageId, path);
 }
 
-Resizer::Resizer(QWidget* parent):
-    QObject(parent)
-{
-    
-}
-
-bool Resizer::eventFilter(QObject* obj, QEvent* event)
-{
-    if (event->type() == QEvent::Resize) {
-        emit resized();
-    }
-
-    return false;
-}
-
 bool VisibilityCatcher::eventFilter(QObject* obj, QEvent* event)
 {
     if (event->type() == QEvent::Show) {

ui/widgets/conversation.h

@@ -23,6 +23,7 @@
 #include <QScopedPointer>
 #include "../../global.h"
 #include "messageline.h"
+#include "../utils/resizer.h"
 
 namespace Ui
 {
@@ -42,18 +43,6 @@ signals:
     void enterPressed();
 };
 
-class Resizer : public QObject {
-    Q_OBJECT
-public:
-    Resizer(QWidget* parent = nullptr);
-    
-protected:
-    bool eventFilter(QObject* obj, QEvent* event) override;
-
-signals:
-    void resized();
-};
-
 class VisibilityCatcher : public QObject {
     Q_OBJECT
 public:

ui/widgets/message.cpp

@@ -17,6 +17,9 @@
  */
 
 #include <QDebug>
+#include <QMimeDatabase>
+#include <QPixmap>
+#include <QFileInfo>
 #include "message.h"
 
 const QRegExp urlReg("^(?!<img\\ssrc=\")((?:https?|ftp)://\\S+)");
@@ -34,10 +37,11 @@ Message::Message(const Shared::Message& source, bool outgoing, const QString& p_
     downloadButton(0),
     file(0),
     progress(0),
-    fileComment(0),
+    fileComment(new QLabel()),
     hasDownloadButton(false),
     hasProgress(false),
-    hasFile(false)
+    hasFile(false),
+    commentAdded(false)
 {
     body->setBackgroundRole(QPalette::AlternateBase);
     body->setAutoFillBackground(true);
@@ -86,6 +90,9 @@ Message::Message(const Shared::Message& source, bool outgoing, const QString& p_
 
 Message::~Message()
 {
+    if (!commentAdded) {
+        delete fileComment;
+    }
 }
 
 QString Message::getId() const
@@ -100,27 +107,23 @@ void Message::setSender(const QString& p_sender)
 
 void Message::addDownloadDialog()
 {
-    if (hasFile) {
-        file->deleteLater();
-        file = 0;
-        hasFile = false;
-    }
-    if (hasProgress) {
-        progress->deleteLater();
-        progress = 0;
-        hasProgress = false;;
-    }
+    hideFile();
+    hideProgress();
     if (!hasDownloadButton) {
+        hideComment();
         if (msg.getBody() == msg.getOutOfBandUrl()) {
             text->setText("");
             text->hide();
         }
         downloadButton = new QPushButton(QIcon::fromTheme("download"), "Download");
-        fileComment = new QLabel(sender->text() + " is offering you to download a file");
+        downloadButton->setToolTip("<a href=\"" + msg.getOutOfBandUrl() + "\">" + msg.getOutOfBandUrl() + "</a>");
+        fileComment->setText(sender->text() + " is offering you to download a file");
+        fileComment->show();
         connect(downloadButton, SIGNAL(clicked()), this, SLOT(onDownload()));
         bodyLayout->insertWidget(2, fileComment);
         bodyLayout->insertWidget(3, downloadButton);
         hasDownloadButton = true;
+        commentAdded = true;
     }
 }
 
@@ -130,54 +133,98 @@ void Message::onDownload()
 }
 
 void Message::setProgress(qreal value)
+{
+    hideFile();
+    hideDownload();
+    if (!hasProgress) {
+        hideComment();
+        if (msg.getBody() == msg.getOutOfBandUrl()) {
+            text->setText("");
+            text->hide();
+        }
+        progress = new QProgressBar();
+        progress->setRange(0, 100);
+        fileComment->setText("Downloading...");
+        fileComment->show();
+        bodyLayout->insertWidget(2, progress);
+        bodyLayout->insertWidget(3, fileComment);
+        hasProgress = true;
+        commentAdded = true;
+    }
+    progress->setValue(value * 100);
+}
+
+void Message::showFile(const QString& path)
+{
+    hideDownload();
+    hideProgress();
+    if (!hasFile) {
+        hideComment();
+        if (msg.getBody() == msg.getOutOfBandUrl()) {
+            text->setText("");
+            text->hide();
+        }
+        QMimeDatabase db;
+        QMimeType type = db.mimeTypeForFile(path);
+        QStringList parts = type.name().split("/");
+        QString big = parts.front();
+        QFileInfo info(path);
+        fileComment = new QLabel();
+        if (big == "image") {
+            file = new Image(path);
+        } else {
+            file = new QLabel();
+            file->setPixmap(QIcon::fromTheme(type.iconName()).pixmap(50));
+            file->setAlignment(Qt::AlignCenter);
+            fileComment->setText(info.fileName());
+            fileComment->setWordWrap(true);
+            fileComment->show();
+        }
+        file->setContextMenuPolicy(Qt::ActionsContextMenu);
+        QAction* openAction = new QAction(QIcon::fromTheme("document-new-from-template"), "Open", file);
+        connect(openAction, &QAction::triggered, [path]() {             //TODO need to get rid of this shame
+            QDesktopServices::openUrl(QUrl::fromLocalFile(path));
+        });
+        file->addAction(openAction);
+        bodyLayout->insertWidget(2, file);
+        bodyLayout->insertWidget(3, fileComment);
+        hasFile = true;
+        commentAdded = true;
+    }
+}
+
+void Message::hideComment()
+{
+    if (commentAdded) {
+        bodyLayout->removeWidget(fileComment);
+        fileComment->hide();
+        fileComment->setWordWrap(false);
+    }
+}
+
+void Message::hideDownload()
+{
+    if (hasDownloadButton) {
+        downloadButton->deleteLater();
+        downloadButton = 0;
+        hasDownloadButton = false;
+    }
+}
+
+void Message::hideFile()
 {
     if (hasFile) {
         file->deleteLater();
         file = 0;
         hasFile = false;
     }
-    if (hasDownloadButton) {
-        downloadButton->deleteLater();
-        fileComment->deleteLater();
-        downloadButton = 0;
-        fileComment = 0;
-        hasDownloadButton = false;
-    }
-    if (!hasProgress) {
-        if (msg.getBody() == msg.getOutOfBandUrl()) {
-            text->setText("");
-            text->hide();
-        }
-        progress = new QLabel(std::to_string(value).c_str());
-        bodyLayout->insertWidget(2, progress);
-        hasProgress = true;
-    }
 }
 
-void Message::showFile(const QString& path)
+void Message::hideProgress()
 {
-    if (hasDownloadButton) {
-        downloadButton->deleteLater();
-        fileComment->deleteLater();
-        downloadButton = 0;
-        fileComment = 0;
-        hasDownloadButton = false;
-    }
     if (hasProgress) {
         progress->deleteLater();
         progress = 0;
-        hasProgress = false;
-    }
-    if (!hasFile) {
-        if (msg.getBody() == msg.getOutOfBandUrl()) {
-            text->setText("");
-            text->hide();
-        }
-        //file = new QLabel("<img src=\"" + path + "\">");
-        file = new QLabel();
-        file->setPixmap(QPixmap(path));
-        //file->setScaledContents(true);
-        bodyLayout->insertWidget(2, file);
-        hasFile = true;
+        hasProgress = false;;
     }
 }

ui/widgets/message.h

@@ -25,8 +25,14 @@
 #include <QLabel>
 #include <QGraphicsDropShadowEffect>
 #include <QPushButton>
+#include <QProgressBar>
+#include <QAction>
+#include <QDesktopServices>
+#include <QUrl>
 
 #include "../../global.h"
+#include "../utils/resizer.h"
+#include "../utils/image.h"
 
 /**
  * @todo write docs
@@ -58,15 +64,21 @@ private:
     QGraphicsDropShadowEffect* shadow;
     QPushButton* downloadButton;
     QLabel* file;
-    QLabel* progress;
+    QProgressBar* progress;
     QLabel* fileComment;
     bool hasDownloadButton;
     bool hasProgress;
     bool hasFile;
+    bool commentAdded;
     
 private slots:
     void onDownload();
-    
+  
+private:
+    void hideDownload();
+    void hideProgress();
+    void hideFile();
+    void hideComment();
 };
 
 #endif // MESSAGE_H