> On Jan. 16, 2013, 10:01 a.m., Thomas Lübking wrote: > > instead of maintoolbar (which could be 128px on even a netbook) maybe > > relate it to the font height? (what seems to be the idea behind 22px) > > Kai Uwe Broulik wrote: > Main toolbar icon size is only settable from 16 to 48 pixels and defaults > to 22. And if you increase the size of toolbar icons, it will make the look > somewhat unified, if this icon grows as well. Don't know how I can properly > relate it to the font height, especially since I cannot just use an arbitrary > number but one of these: 16, 22, 32, 48, .. > > Thomas Lübking wrote: > > Main toolbar icon size is only settable from 16 to 48 pixels and > defaults to 22. > Arbtrary GUI config limitation: > kwriteconfig --file kdeglobals --group MainToolbarIcons --key Size 128 > > > And if you increase the size of toolbar icons, it will make the look > somewhat unified > As mentioned, i don't know about the idea behind this, just pointing that > a fixed size usually near the font height was initially picked. > If you raise dpi, thus font pixels ... you get what i mean. > > > Don't know how I can properly relate it to the font height > QFontMetrics(QWidget::font()).height() > > QPixmap QIcon::pixmap ( int w, int h, Mode mode = Normal, State state = > Off ) const > This is an overloaded function. > Returns a pixmap of size QSize(w, h). The pixmap might be smaller than > requested, but never larger. > > You /can/ use arbitrary sizes but may want to match them to the closest > size (simple for the 2^n sizes, but there's still stupid 22px - as it's > apparently not a hot path, just move up until qAbs(height-n) gets bigger > again)
> Arbtrary GUI config limitation: But not my problem if a user bypasses this :P > QFontMetrics(QWidget::font()).height() Thanks! Ah, this is how to use this. Already knew that there was this QFontMetrics thing but didn't know how to use. Will make my work on other parts easier as well. > You /can/ use arbitrary sizes but may want to match them to the closest size Yes, I know, but wanted to point out that I didn't want this to avoid blurry icons ;) I don't understand this 22 thing anyway, why not just 24? Who knows.. - Kai Uwe ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108434/#review25636 ----------------------------------------------------------- On Jan. 16, 2013, 1:34 a.m., Kai Uwe Broulik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/108434/ > ----------------------------------------------------------- > > (Updated Jan. 16, 2013, 1:34 a.m.) > > > Review request for kdelibs. > > > Description > ------- > > It took me hours (really, it's like. System Settings adds a KCModuleProxy > which calls KCModuleLoader to load a KCModule which is in some different > directory which uses KPageDialog (where I looked at first …) which uses > KPageWidget which is a private class inside which then uses KPageView bla.. > :D) > > Finally got it and made it use the MainToolbar size which also defaults to > 22px, so you won't notice any difference, but users of MacBook Pro Retina and > similar devices will appreciate. Also makes it unified a bit because it is > always the toolbar icon size. > > > Diffs > ----- > > kdeui/paged/kpageview.cpp 8863934 > > Diff: http://git.reviewboard.kde.org/r/108434/diff/ > > > Testing > ------- > > Yup, see screenshots. > > Only minor problem is that it doesn't react to when the icon size changes > (like buttons and other elements do). > > > File Attachments > ---------------- > > KPageDialog icon adapted > http://git.reviewboard.kde.org/media/uploaded/files/2013/01/16/iconding.png > > > Thanks, > > Kai Uwe Broulik > >
