Bug 353081

Summary: Remember wallet manager window size / At least increase default
Product: [Frameworks and Libraries] frameworks-kwallet Reporter: Ralf Jung <post>
Component: generalAssignee: Valentin Rusu <valir>
Status: REOPENED ---    
Severity: normal CC: kdelibs-bugs-null
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description Ralf Jung 2015-09-23 11:15:53 UTC
The wallet manager always appears with a window size that's too small to read the content of wallets, so I always have to manually increase the window size. Every single time.

Reproducible: Always

Steps to Reproduce:
1. Start kwalletmanager
2. Open a wallet
3. Select some entry

Actual Results:  
The window is so small that the entry can hardly be read.

Expected Results:  
The window should be large enough such that at least 10 lines of the entry are visible. I'm on a full-HD screen, there's *so* much unused screen space around this window...

The wallet manager could either remember the size it had last time, and restore that size - or at the very least, the default should be large enough to be usable. Insisting on an unusably small window size is getting in the user's way of actually using the program.
Comment 1 Valentin Rusu 2015-11-28 15:26:13 UTC
Git commit 917c49e4b8d46970bfe76f87ffc845ce8d5b9fe1 by Valentin Rusu.
Committed on 15/11/2015 at 22:14.
Pushed by vrusu into branch 'master'.

Using a bigger default size

Couldn't test this though, as I'm using a tiling window manager that helped me
definitely forget about the window positioning stuff.

M  +2    -2    src/manager/walletcontrolwidget.ui

http://commits.kde.org/kwallet/917c49e4b8d46970bfe76f87ffc845ce8d5b9fe1
Comment 2 Ralf Jung 2016-03-28 11:36:53 UTC
Has this patch landed in time for the 15.12 release? I upgraded kwalletmanager to 15.12 (Debian packages), but the window is still too small per default, nor does it remember its size.
Comment 3 Ralf Jung 2016-03-28 12:34:24 UTC
Actually, your patch does not even help here. However, this one does:

--- a/src/manager/kwalletmanager.cpp
+++ b/src/manager/kwalletmanager.cpp
@@ -179,6 +179,8 @@
     setupGUI(Keys | Save | Create, QLatin1String("kwalletmanager.rc"));
     setStandardToolBarMenuEnabled(false);
 
+    resize(900, 720);
+
     if (_tray) {
 //        _tray->show();
     } else {