Bug 263601 - many windows bigger than screen and not scrollable
Summary: many windows bigger than screen and not scrollable
Status: VERIFIED WORKSFORME
Alias: None
Product: Kleopatra Mobile
Classification: Unmaintained
Component: general (show other bugs)
Version: unspecified
Platform: Windows CE Microsoft Windows CE
: HI normal
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-01-19 02:09 UTC by Marcus Brinkmann
Modified: 2011-02-15 19:16 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:

Attachments
screenshots (95.30 KB, application/zip)
2011-01-19 16:37 UTC, Andreas Holzammer 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
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.