From 17b5cc32dc56552ac3ae7285f7ffdb194b907ffd Mon Sep 17 00:00:00 2001
From: Adasauce <adasauce@with.parts>
Date: Fri, 14 Feb 2020 16:24:53 -0400
Subject: [PATCH] use the standard system download path for downloads
QFileDialog's dir arg (which was set to the incoming file name from the
Matrix download) can take a full path to suggest. By prepending
QStandardPaths::DownloadLocation, it opens to the system's download
folder and proposes the filename as the download name.
Using QStandardPaths should make this work on other platforms, and from
what I read, its possible for this to return an empty string on
platforms where it doesn't support it, so this should essentially
revert to the previous functionality if Qt can't determine the system's
download location.
---
src/timeline/TimelineModel.cpp | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/src/timeline/TimelineModel.cpp b/src/timeline/TimelineModel.cpp
index f084b4dde..625377fbf 100644
--- a/src/timeline/TimelineModel.cpp
+++ b/src/timeline/TimelineModel.cpp
@@ -1298,10 +1298,12 @@ TimelineModel::saveMedia(QString eventId) const
dialogTitle = tr("Save file");
}
- QString filterString = QMimeDatabase().mimeTypeForName(mimeType).filterString();
+ const QString filterString = QMimeDatabase().mimeTypeForName(mimeType).filterString();
+ const QString downloadsFolder = QStandardPaths::writableLocation(QStandardPaths::DownloadLocation);
+ const QString openLocation = downloadsFolder + "/" + originalFilename;
- auto filename = QFileDialog::getSaveFileName(
- manager_->getWidget(), dialogTitle, originalFilename, filterString);
+ const QString filename = QFileDialog::getSaveFileName(
+ manager_->getWidget(), dialogTitle, openLocation, filterString);
if (filename.isEmpty())
return;
--
GitLab