----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120155/#review66332 -----------------------------------------------------------
Use KPixmapSequence if you need to access an FDO animation icon. - Christoph Feck On Sept. 12, 2014, 8:35 a.m., Dan Vrátil wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/120155/ > ----------------------------------------------------------- > > (Updated Sept. 12, 2014, 8:35 a.m.) > > > Review request for KDE Frameworks. > > > Repository: kiconthemes > > > Description > ------- > > KIconLoader (and KIconTheme) were always using single int to specify icon > size, which implies only square icons. When requesting non-square icons > however (such as pixmap sequence animations), the returned QPixmap is > deformed. This patch ports all internals of KIconLoader to QSize. > > > Diffs > ----- > > src/kiconengine.cpp 530403e > src/kiconloader.h 46d3017 > src/kiconloader.cpp 2587b46 > src/kicontheme.h ca04879 > src/kicontheme.cpp 4f0e522 > > Diff: https://git.reviewboard.kde.org/r/120155/diff/ > > > Testing > ------- > > QIcon icon = QIcon::fromTheme(QLatin1String("process-working")); > QPixmap pix = icon.pixmap(22, 8 * 22); > > With this patch, the pix looks as expected. > > > Thanks, > > Dan Vrátil > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
