----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107357/#review24306 -----------------------------------------------------------
I was just going to apply the diff but realized with the new code we get "label buddies" and that makes the _ appear in almost all the labels of the first column that makes no sense as you can't use Alt+key to focus on the buddy since the buddy is also a label that can't hold focus. Can you please try to make those not appear? - Albert Astals Cid On Nov. 17, 2012, 7:51 p.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, 7:51 p.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 > >
_______________________________________________ Okular-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/okular-devel
