----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106855/#review20732 -----------------------------------------------------------
Fix those below and ship it from me ;) contact-delegate-compact.cpp <http://git.reviewboard.kde.org/r/106855/#comment16372> This needs to be KIconLoader::Toolbar, which is 22 by default. contact-delegate-compact.cpp <http://git.reviewboard.kde.org/r/106855/#comment16373> Same here - KIconLoader::Toolbar contact-delegate.cpp <http://git.reviewboard.kde.org/r/106855/#comment16374> Avatar size here is ok, presence icon however must be KIconLoader::Toolbar. - Martin Klapetek On Oct. 21, 2012, 3:07 p.m., Michael Zanetti wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/106855/ > ----------------------------------------------------------- > > (Updated Oct. 21, 2012, 3:07 p.m.) > > > Review request for Telepathy and David Edmundson. > > > Description > ------- > > Right now the ktp-contact-list has some hard coded values for sizes which > makes it very hard to use on high-res screens. > > This patch uses KIconLoader to paint the icons using the sizes configured in > the systemsettings instead of the hardcoded values. > > > Diffs > ----- > > contact-delegate-compact.cpp 1915397 > contact-delegate.h 8508f7e > contact-delegate.cpp f79b1d5 > contact-overlays.cpp 8ea1b1a > > Diff: http://git.reviewboard.kde.org/r/106855/diff/ > > > Testing > ------- > > > Thanks, > > Michael Zanetti > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
