[Okular-devel] Review Request: Properties dialog: switch to QFormLayout and showing icon for mime type

[Thomas Fischer]

> On Nov. 17, 2012, 2:14 p.m., Albert Astals Cid wrote:
> > The QFormLayout is not needed,  that's a class totally tailored for small screen state devices (phones) and it's not going to happen that we port this dialog to a phone.
> > 
> > The icon thing looks like a nice addition and i don't object to the showing of the comment either,

> The QFormLayout is not needed, that's a class
> totally tailored for small screen state devices (phones)
According to the documentation, this is only one argument to use this class. The other one is that it adepts to the desktop's settings (KDE, Gnome, Windows, Mac) regarding layout alignment. In this respect, it is an improvement over the previous addWidget(..., Qt::AlignRight);
Finally, the addRow statement takes care of creating the label instead of creating and layouting QLabel *key manually.

> The icon thing looks like a nice addition and
> i don't object to the showing of the comment either
Working on that ...


- Thomas


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107357/#review22121
-----------------------------------------------------------


On Nov. 17, 2012, 10:56 a.m., Thomas Fischer wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107357/
> -----------------------------------------------------------
> 
> (Updated Nov. 17, 2012, 10:56 a.m.)
> 
> 
> Review request for Okular.
> 
> 
> Description
> -------
> 
> This patch changes the properties dialog to use QFormLayout instead of a two-column QGridLayout. This should make the dialog more compatible with other user interfaces.
> For the mime type row, instead of just showing the mime type's name, a small QLabel is used to show the mime type's icon.
> 
> Maybe it would be a good idea to not only show the mime type's name, but also the description (KMimeType::comment(..)). Example: "PDF Document (application/pdf)"
> 
> 
> Diffs
> -----
> 
>   ui/propertiesdialog.cpp 2ef8220 
> 
> Diff: http://git.reviewboard.kde.org/r/107357/diff/
> 
> 
> Testing
> -------
> 
> 
> Screenshots
> -----------
> 
> QFormLayout and mime type icon
>   http://git.reviewboard.kde.org/r/107357/s/835/
> 
> 
> Thanks,
> 
> Thomas Fischer
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20121117/edc574d4/attachment-0001.html>