New geometry only saved when main window is closed

[stefanb2]

Try this

• start Qt-SESAM
• resize main window
• system tray icon menu "Quit" to exit program
• start Qt-SESAM

It will still show the old main window geometry.

If you CLOSE the main window and then exit the application, it works fine.

[607011]

Oops! I'll check on that.

[stefanb2]

$ fgrep geometry .config/ct/QtSESAM.ini
geometry="@ByteArray(\x1\xd9\xd0\xcb\0\x2\0\0\0\0\b7\0\0\0/\0\0\f\x8e\0\0\x3g\0\0\b;\0\0\0L\0\0\f\xed\0\0\x3j\0\0\0\x1\0\0\0\0\a\x80)"

Resize window (-> same values)

$ fgrep geometry .config/ct/QtSESAM.ini
geometry="@ByteArray(\x1\xd9\xd0\xcb\0\x2\0\0\0\0\b7\0\0\0/\0\0\f\x8e\0\0\x3g\0\0\b;\0\0\0L\0\0\f\xed\0\0\x3j\0\0\0\x1\0\0\0\0\a\x80)"

Close window (-> new values)

$ fgrep geometry .config/ct/QtSESAM.ini
geometry=@ByteArray(\x1\xd9\xd0\xcb\0\x2\0\0\0\0\b\xfd\0\0\0\xcc\0\0\r\xb7\0\0\x4\v\0\0\t\x1\0\0\0\xe9\0\0\r\xb3\0\0\x4\a\0\0\0\x1\0\0\0\0\a\x80)

I guess you would need to explicitly call close window on the main window when exiting the program?

[607011]

The Windows version behaves as expected: Regardless of whether you close the application via File/Exit from the main menu or by selecting Quit from the tray icon, the geometry is saved and restored correctly.

Weird!

[stefanb2]

This is side-effect of issue #44 and not working close() slot. Closing this one.

[stefanb2]

Actually let's keep issue #44 open and close this one for fixing the Exit issue.

[stefanb2]

From 3d86619d385a35604e9821b53891b6ad87560952 Mon Sep 17 00:00:00 2001
From: Stefan Becker <chemobejk@gmail.com>
Date: Wed, 16 Sep 2015 20:25:04 +0300
Subject: [PATCH] mainwindow: hide tray icon in closeEvent()

At least on Linux (or KDE Plasma?) the application must hide the system
tray icon in main window closeEvent() in order for Qt to shut down the
application.

Resolves issue #45
---
 Qt-SESAM/mainwindow.cpp | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Qt-SESAM/mainwindow.cpp b/Qt-SESAM/mainwindow.cpp
index 05f8a52..0a782b2 100644
--- a/Qt-SESAM/mainwindow.cpp
+++ b/Qt-SESAM/mainwindow.cpp
@@ -320,6 +320,7 @@ void MainWindow::closeEvent(QCloseEvent *e)
   auto prepareExit = [this]() {
     d_ptr->masterPasswordDialog->close();
     d_ptr->optionsDialog->close();
+    d_ptr->trayIcon.hide();
     saveSettings();
     invalidatePassword(false);
     d_ptr->keyGenerationFuture.waitForFinished();
-- 
2.4.3

[607011]

Resolved by commit ceda75e