From 4500bcd5779e87d553c1e5fbb409ae6d80bec086 Mon Sep 17 00:00:00 2001
From: Nicolas Werner <nicolas.werner@hotmail.de>
Date: Thu, 20 Feb 2020 18:11:30 +0100
Subject: [PATCH] Use spacing for layouting settings

---
 src/UserSettingsPage.cpp | 32 ++++++--------------------------
 src/UserSettingsPage.h   |  1 -
 2 files changed, 6 insertions(+), 27 deletions(-)

diff --git a/src/UserSettingsPage.cpp b/src/UserSettingsPage.cpp
index 63dcffbd1..2cac783c2 100644
--- a/src/UserSettingsPage.cpp
+++ b/src/UserSettingsPage.cpp
@@ -331,9 +331,14 @@ UserSettingsPage::UserSettingsPage(QSharedPointer<UserSettings> settings, QWidge
 
         QScroller::grabGesture(scrollArea_, QScroller::TouchGesture);
 
+        auto spacingAroundForm = new QHBoxLayout;
+        spacingAroundForm->addStretch(1);
+        spacingAroundForm->addLayout(formLayout_, 0);
+        spacingAroundForm->addStretch(1);
+
         auto scrollAreaContents_ = new QWidget{this};
         scrollAreaContents_->setObjectName("UserSettingScrollWidget");
-        scrollAreaContents_->setLayout(formLayout_);
+        scrollAreaContents_->setLayout(spacingAroundForm);
 
         scrollArea_->setWidget(scrollAreaContents_);
         topLayout_->addLayout(topBarLayout_);
@@ -432,31 +437,6 @@ UserSettingsPage::showEvent(QShowEvent *)
           utils::humanReadableFingerprint(olm::client()->identity_keys().ed25519));
 }
 
-void
-UserSettingsPage::resizeEvent(QResizeEvent *event)
-{
-        auto preWidth = width();
-
-        // based on the width of the widest item currently in the layout
-        // deviceFingerprintValue_ used for recalculating the margins of
-        // the formLayout_ on resize to help with small screens and mobile devices.
-
-        double minFormWidth = deviceFingerprintValue_->width();
-
-        if (preWidth * 0.5 > minFormWidth)
-                sideMargin_ = preWidth * 0.25;
-        else
-                sideMargin_ = static_cast<double>(preWidth - minFormWidth) / 2.;
-
-        if (sideMargin_ < 60)
-                sideMargin_ = 0;
-
-        formLayout_->setContentsMargins(
-          sideMargin_, LayoutTopMargin, sideMargin_, LayoutBottomMargin);
-
-        QWidget::resizeEvent(event);
-}
-
 void
 UserSettingsPage::paintEvent(QPaintEvent *)
 {
diff --git a/src/UserSettingsPage.h b/src/UserSettingsPage.h
index ef8258b61..a1b7b0841 100644
--- a/src/UserSettingsPage.h
+++ b/src/UserSettingsPage.h
@@ -152,7 +152,6 @@ public:
 
 protected:
         void showEvent(QShowEvent *event) override;
-        void resizeEvent(QResizeEvent *event) override;
         void paintEvent(QPaintEvent *event) override;
 
 signals:
-- 
GitLab