----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118561/#review59426 -----------------------------------------------------------
This review has been submitted with commit f23fb35f2daed4671089c70f46356d74a59b6840 by Martin Klapetek to branch master. - Commit Hook On June 6, 2014, 12:02 p.m., Martin Klapetek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/118561/ > ----------------------------------------------------------- > > (Updated June 6, 2014, 12:02 p.m.) > > > Review request for KDE Frameworks and Christoph Feck. > > > Repository: kiconthemes > > > Description > ------- > > KIconTheme in kdelibs4 was searching KGlobal::dirs()->resourceDirs("data") to > find all icons; the status notifier dataengine was then setting custom > resourceDirs(..) with custom SNI theme paths (which the SNIs can pass) and so > the SNI icons always ended up in the theme search paths (the SNI icons are > stored as a whole theme). > > With the port to QStandardPaths, we lost the ability to pass custom dirs into > the theme search paths and that results in status notifier icons in Plasma > Next having "unknown" icons as their icon theme paths are not searched and so > icons are not found. This is the case mostly of the Qt4 apps running on Qt > with the QSystrayIcon-to-SNI-patches (case of *buntu and I heard opensuse > too?) > > KIconLoader however has "addAppDir(..)" method, so I expanded that method to > actually take an "app dir" and add it to the theme search paths. Plasma Next > now have proper icons in the systray. > > > Diffs > ----- > > src/kiconloader.h 68ccd09 > src/kiconloader.cpp 4080a1d > src/kicontheme.h 73011e2 > src/kicontheme.cpp 12337e8 > > Diff: https://git.reviewboard.kde.org/r/118561/diff/ > > > Testing > ------- > > > Thanks, > > Martin Klapetek > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
