Limit window size to 2/3

Fixes: #8366

Author: TheOneRing

changelog/unreleased/8366

@@ -0,0 +1,6 @@
+Bugfix: Limit min window size to 2/3 available
+
+When scaling was used the window could become bigger than the screen.
+The size is now limited to 2/3 of the screen.
+
+https://github.com/owncloud/client/issues/8366

src/gui/settingsdialog.cpp

@@ -30,21 +30,23 @@
 #include "protocolwidget.h"
 #include "owncloudsetupwizard.h"
 
+#include <QImage>
 #include <QLabel>
-#include <QStandardItemModel>
-#include <QStackedWidget>
+#include <QLayout>
+#include <QMessageBox>
+#include <QPainter>
+#include <QPainterPath>
+#include <QPixmap>
 #include <QPushButton>
+#include <QScreen>
 #include <QSettings>
+#include <QStackedWidget>
+#include <QStandardItemModel>
 #include <QToolBar>
 #include <QToolButton>
-#include <QLayout>
 #include <QVBoxLayout>
-#include <QPixmap>
-#include <QImage>
 #include <QWidgetAction>
-#include <QPainter>
-#include <QPainterPath>
-#include <QMessageBox>
+#include <QWindow>
 
 #ifdef Q_OS_MAC
 #include "settingsdialog_mac.h"
@@ -250,11 +252,7 @@ SettingsDialog::SettingsDialog(ownCloudGui *gui, QWidget *parent)
     customizeStyle();
 
     cfg.restoreGeometry(this);
-    if (size().width() < minimumSizeHint().width() || size().height() < minimumSizeHint().height())
-    {
-        resize(minimumSizeHint());
-    }
-
+    setMinimumSize(minimumSizeHint());
 #ifdef Q_OS_MAC
     setActivationPolicy(ActivationPolicy::Accessory);
 #endif
@@ -267,7 +265,20 @@ SettingsDialog::~SettingsDialog()
 
 QSize SettingsDialog::minimumSizeHint() const
 {
-    return {800, 500};
+    const auto screen = windowHandle() ? windowHandle()->screen() : QApplication::screenAt(QCursor::pos());
+    const auto geometry = screen->availableSize();
+    const QSize min { 800, 600 };
+    const QSize currentMin { static_cast<int>(geometry.width() * 0.75), static_cast<int>(geometry.height() * 0.75) };
+    if (currentMin.width() < min.width() || currentMin.height() < min.height()) {
+        return currentMin;
+    }
+    return min;
+}
+
+QSize SettingsDialog::sizeHintForChild() const
+{
+    auto size = ocApp()->gui()->settingsDialog()->size();
+    return { static_cast<int>(size.width() * 0.9), static_cast<int>(size.height() * 0.9) };
 }
 
 QWidget* SettingsDialog::currentPage()

src/gui/settingsdialog.h

@@ -51,8 +51,10 @@ class SettingsDialog : public QMainWindow
 
 public:
     explicit SettingsDialog(ownCloudGui *gui, QWidget *parent = nullptr);
-    ~SettingsDialog() override;    
+    ~SettingsDialog() override;
+
     QSize minimumSizeHint() const override;
+    QSize sizeHintForChild() const;
 
     QWidget* currentPage();
 

src/gui/sharedialog.cpp

@@ -137,8 +137,7 @@ ShareDialog::ShareDialog(QPointer<AccountState> accountState,
     connect(job, &PropfindJob::finishedWithError, this, &ShareDialog::slotPropfindError);
     job->start();
 
-    auto size = ocApp()->gui()->settingsDialog()->minimumSizeHint();
-    resize(size.width() - 50, size.height() - 50);
+    setMinimumSize(minimumSizeHint());
 }
 
 ShareDialog::~ShareDialog()
@@ -219,6 +218,11 @@ void ShareDialog::showSharingUi()
     }
 }
 
+QSize ShareDialog::minimumSizeHint() const
+{
+    return ocApp()->gui()->settingsDialog()->sizeHintForChild();
+}
+
 void ShareDialog::slotThumbnailFetched(const int &statusCode, const QByteArray &reply)
 {
     if (statusCode != 200) {

src/gui/sharedialog.h

@@ -48,6 +48,8 @@ class ShareDialog : public QDialog
         QWidget *parent);
     ~ShareDialog() override;
 
+    QSize minimumSizeHint() const override;
+
 private slots:
     void done(int r) override;
     void slotPropfindReceived(const QMap<QString, QString> &result);

src/gui/sharedialog.ui

@@ -2,14 +2,6 @@
 <ui version="4.0">
  <class>OCC::ShareDialog</class>
  <widget class="QDialog" name="OCC::ShareDialog">
-  <property name="geometry">
-   <rect>
-    <x>0</x>
-    <y>0</y>
-    <width>408</width>
-    <height>281</height>
-   </rect>
-  </property>
   <property name="windowTitle">
    <string>Dialog</string>
   </property>

src/gui/wizard/owncloudwizard.cpp

@@ -54,8 +54,6 @@ OwncloudWizard::OwncloudWizard(QWidget *parent)
     , _credentialsPage(nullptr)
 {
     setObjectName("owncloudWizard");
-    auto size = ocApp()->gui()->settingsDialog()->minimumSizeHint();
-    resize(size.width() - 50, size.height() - 50);
 
     setWindowFlags(windowFlags() & ~Qt::WindowContextHelpButtonHint);
     setPage(WizardCommon::Page_ServerSetup, _setupPage);
@@ -89,6 +87,8 @@ OwncloudWizard::OwncloudWizard(QWidget *parent)
     setOption(QWizard::CancelButtonOnLeft);
     setTitleFormat(Qt::RichText);
     setSubTitleFormat(Qt::RichText);
+
+    setMinimumSize(minimumSizeHint());
 }
 
 void OwncloudWizard::setAccount(AccountPtr account)
@@ -276,4 +276,9 @@ void OwncloudWizard::askExperimentalVirtualFilesFeature(QWidget *receiver, const
     msgBox->open();
 }
 
+QSize OCC::OwncloudWizard::minimumSizeHint() const
+{
+    return ocApp()->gui()->settingsDialog()->sizeHintForChild();
+}
+
 } // end namespace

src/gui/wizard/owncloudwizard.h

@@ -72,6 +72,8 @@ class OwncloudWizard : public QWizard
 
     DetermineAuthTypeJob::AuthType authType() const;
 
+    QSize minimumSizeHint() const override;
+
 public slots:
     void setAuthType(DetermineAuthTypeJob::AuthType type);
     void setRemoteFolder(const QString &);