diff options
author | Michael Schuster <michael@schuster.ms> | 2020-10-22 04:57:29 +0300 |
---|---|---|
committer | Kevin Ottens (Rebase PR Action) <er-vin@users.noreply.github.com> | 2020-10-22 14:29:38 +0300 |
commit | 0fb10163a3b8ae08f0c149570a3c0705734c3511 (patch) | |
tree | 5f238aa5d0a0f9cb5513e4b3a1b3f1339d3069c3 /src/gui/owncloudgui.cpp | |
parent | 40aa53bf7656e68a440bc649597957e39908f13c (diff) |
Delay initialization of SettingsDialog
ownCloudGui::slotShowSettings already got what it takes to create it only when we try to show it for the first time.
This however has some implications:
Pros:
- Only created when needed, while testing saved ca. 20 MB of RAM and got freed again after closing the dialog.
- Since we defaulted to the new Tray UI from 3.0, this is an added bonus for users don't opening the settings.
Cons:
- Resources like the avatar image have to be refetched everytime the dialog is recreated.
This may be desired as well, because it ensures displaying no outdated info (e.g. on connection issues).
Signed-off-by: Michael Schuster <michael@schuster.ms>
Diffstat (limited to 'src/gui/owncloudgui.cpp')
-rw-r--r-- | src/gui/owncloudgui.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/gui/owncloudgui.cpp b/src/gui/owncloudgui.cpp index f328cb79d..9b8c42a7b 100644 --- a/src/gui/owncloudgui.cpp +++ b/src/gui/owncloudgui.cpp @@ -59,7 +59,7 @@ const char propertyAccountC[] = "oc_account"; ownCloudGui::ownCloudGui(Application *parent) : QObject(parent) , _tray(nullptr) - , _settingsDialog(new SettingsDialog(this)) + , _settingsDialog(nullptr) , _logBrowser(nullptr) #ifdef WITH_LIBCLOUDPROVIDERS , _bus(QDBusConnection::sessionBus()) |