----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107357/ -----------------------------------------------------------
(Updated Nov. 17, 2012, 5:09 p.m.) Review request for Okular. Changes ------- Instead of writing out the "raw" mimetype (example: "application/pdf"), it shows a description of the mime type followed by the mime type's name in parentheses (example: "PDF document (application/pdf)"). 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 (updated) ----- 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
_______________________________________________ Okular-devel mailing list Okular-devel@kde.org https://mail.kde.org/mailman/listinfo/okular-devel
