Use MainWindow instead of QApplication::activeWindow to reference the main window

The latter will not always return a valid pointer.

fixes #363
pull/1/head
Konstantinos Sideris 7 years ago
parent 31d09dbd68
commit fde066e236
  1. 2
      src/CommunitiesList.cc
  2. 14
      src/dialogs/PreviewUploadOverlay.cc

@ -1,5 +1,5 @@
#include "Cache.h"
#include "CommunitiesList.h"
#include "Cache.h"
#include "Logging.hpp"
#include "MatrixClient.h"

@ -25,6 +25,7 @@
#include "Config.h"
#include "Logging.hpp"
#include "MainWindow.h"
#include "Utils.h"
#include "dialogs/PreviewUploadOverlay.h"
@ -61,9 +62,16 @@ PreviewUploadOverlay::PreviewUploadOverlay(QWidget *parent)
void
PreviewUploadOverlay::init()
{
auto window = QApplication::activeWindow();
auto winsize = window->frameGeometry().size();
auto center = window->frameGeometry().center();
QSize winsize;
QPoint center;
auto window = MainWindow::instance();
if (window) {
winsize = window->frameGeometry().size();
center = window->frameGeometry().center();
} else {
nhlog::ui()->warn("unable to load the retrieve MainWindow's size");
}
fileName_.setText(QFileInfo{filePath_}.fileName());

Loading…
Cancel
Save