Bug 263601

Summary: many windows bigger than screen and not scrollable
Product: [Applications] Kleopatra Mobile Reporter: Marcus Brinkmann <marcus.brinkmann>
Component: generalAssignee: kdepim bugs <kdepim-bugs>
Status: VERIFIED WORKSFORME    
Severity: normal CC: aheinecke, andreas.holzammer, ludwig.reiter, tokoe
Priority: HI    
Version: unspecified   
Target Milestone: ---   
Platform: Windows CE   
OS: Microsoft Windows CE   
Latest Commit: Version Fixed In:
Attachments: screenshots

Description Marcus Brinkmann 2011-01-19 02:09:01 UTC
Version:           unspecified (using Devel) 
OS:                Windows CE

some windows are bigger than the screen and not scrollable.  I can move it around a bit (mostly to left and right) but that doesn't show me the window bottom.  Among these are:

The certificate details window.

The owner trust setting.

The expiry date window (a tiny bit).


Reproducible: Always
Comment 1 Andreas Holzammer 2011-01-19 16:34:30 UTC
also there are many too big windows with scrollbars in create certificate, as seen in the screenshots.
Comment 2 Andreas Holzammer 2011-01-19 16:37:50 UTC
Created attachment 56195 [details]
screenshots
Comment 3 Bernhard E. Reiter 2011-01-21 12:03:42 UTC
Raising priority as this might affect a major feature (crypto) significantly.
Comment 4 Marcus Brinkmann 2011-01-21 14:59:43 UTC
FWIW, I agree that the priority is high for the certificate details, which are important to see.

The owner trust feature is an advanced feature IMO that one usually does not need to touch.  So here the priority is not high IMO.

The calendar issue is only cosmetic: It is still possible to enter all dates.
Comment 5 Tobias Koenig 2011-01-31 16:36:41 UTC
Git commit 03821ea6bab237aab6efbcf33d418020a58d7439 by Tobias Koenig.
Pushed by tokoe into branch 'master'.

Use KDialog as base class to fix layout on WinCE

Let CertificateDetailsDialog, ExpiryDialog and OwnerTrustDialog
inherit KDialog instead of QDialog to make them usable under WinCE
where KDialog adds a QScrollArea automatically if the dialog size
is larger than the screen.

BUG: 263601

M  +3    -2    kleopatra/dialogs/certificatedetailsdialog.cpp     
M  +2    -2    kleopatra/dialogs/certificatedetailsdialog.h     
M  +137  -202  kleopatra/dialogs/certificatedetailsdialog.ui     
M  +3    -2    kleopatra/dialogs/expirydialog.cpp     
M  +2    -2    kleopatra/dialogs/expirydialog.h     
M  +50   -91   kleopatra/dialogs/expirydialog.ui     
M  +8    -5    kleopatra/dialogs/ownertrustdialog.cpp     
M  +2    -2    kleopatra/dialogs/ownertrustdialog.h     
M  +177  -231  kleopatra/dialogs/ownertrustdialog.ui     

http://commits.kde.org/ee6cc38b/03821ea6bab237aab6efbcf33d418020a58d7439
Comment 6 Ludwig Reiter 2011-02-04 14:01:02 UTC
version 2011-03-02 16:36

Test:
1. Start kleopatra
2. Click on a certificate.
But the size of the appearing dialog is not fitting the page.
Reopened
Comment 7 Tobias Koenig 2011-02-07 12:52:15 UTC
Hej Ludwig,

the dialog does not match on the screen but you have scroll bars now to reach each part of the dialog. This was not possible previously.
The ideal solution would be to rewrite the GUI to make it mobile aware, but this is out-of-scope for this project. We just made it possible to access every functionality of Kleopatra with the additional scroll bars.

Ciao,
Tobias
Comment 8 Andre Heinecke 2011-02-15 19:16:07 UTC
I can verify that the certificate details can be read, its ugly but ok.