----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108442/#review33572 -----------------------------------------------------------
kio/kfile/kpropertiesdialog.cpp <http://git.reviewboard.kde.org/r/108442/#comment24810> 18*dpiRatio + 2*iconButton->style()... ? The code looks suspisious enough, though (the button should have a sanme resizePolicy and adjust to the loaded icon automatically, yesno?) *shrug* - Thomas Lübking On May 15, 2013, 7:37 p.m., Kai Uwe Broulik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/108442/ > ----------------------------------------------------------- > > (Updated May 15, 2013, 7:37 p.m.) > > > Review request for kdelibs. > > > Description > ------- > > The KIconButton and the other occurences assume that the icon size for > desktop icons is always 48x48. This assumption is wrong. > This patch makes KPropertiesDialog use the proper IconSize. > > There are other places that need fixing too (eg. Dolphin's Place edit dialog > or KMenuEdit) which I will fix later as well. > > So, with KDE Frameworks at the horizon and kdelibs frozen, does this mean, > when I am re-writing the KIconDialog to be more userfriendly, use an UI file, > introduce new strings, etc this cannot go into master but only frameworks > branch? > > > Diffs > ----- > > kio/kfile/kicondialog.cpp 73a7449 > kio/kfile/kpropertiesdialog.cpp b4cd8ee > > Diff: http://git.reviewboard.kde.org/r/108442/diff/ > > > Testing > ------- > > Yup, you won't notice any difference with default settings but with higher > icon size and font scales perfectly and looks good. See screenshot. > > > File Attachments > ---------------- > > KPropertiesDialog with Retina settings > > http://git.reviewboard.kde.org/media/uploaded/files/2013/01/16/iconbutton.png > > > Thanks, > > Kai Uwe Broulik > >
