> On May 2, 2016, 1:17 p.m., Sebastian Kügler wrote: > > src/kiconloader.cpp, line 869 > > <https://git.reviewboard.kde.org/r/127779/diff/9/?file=461529#file461529line869> > > > > Where does this one get deleted? > > Marco Martin wrote: > it's a QScopedPointer, so it gets deleted when the function returns
Ah right ... sorry for being blind. :) - Sebastian ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127779/#review95076 ----------------------------------------------------------- On May 4, 2016, 9:56 a.m., Marco Martin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/127779/ > ----------------------------------------------------------- > > (Updated May 4, 2016, 9:56 a.m.) > > > Review request for KDE Frameworks and Plasma. > > > Repository: kiconthemes > > > Description > ------- > > Breeze uses stylesheet information to color its icons with some "named" > colors that change depending from the current system color scheme, this is > already used since some time in the icons used by the Plasma shell. > But QWidget applications have the same problem, when the user changes the > color scheme from breeze to breeze dark (or any color scheme), most of the > icons will be black on black. > This patch clones (a bit simplified and taking only the most "important" > colors) the logic used by Plasma::Svg to color icons with the stylesheet. > > even tough it's doing more things at icon generation, an application that > uses a lot of icons like Dolphin doesn't seem to have noticeable startup time > difference, even when the image cache is not present yet, so i hope is not an > unacceptable performance tradeoff (successive loads are unchanged as are from > the image cache). > > still some questions and things that can be optimized, like > > * an optional key in the theme metadata file to explicitly enable this, to > not have it running in themes that don't care? > > * can i use karchive in this framework?, so it would work with svgz icons as > well > > * right now to refresh icons at runtime it depends from a patch in the colors > kcm to emit iconchanges as well, alternatively kiconloader could watch for > kcolorscheme changes as well, but i don't think is strictly necessary > > > Diffs > ----- > > CMakeLists.txt 2e838e8 > autotests/coloredsvgicon.svg PRE-CREATION > autotests/kiconloader_unittest.cpp 0e47cc8 > autotests/resources.qrc a19c963 > src/CMakeLists.txt 0e30a35 > src/kiconloader.cpp 75ab482 > src/kicontheme.h 3190665 > src/kicontheme.cpp 0996054 > > Diff: https://git.reviewboard.kde.org/r/127779/diff/ > > > Testing > ------- > > > File Attachments > ---------------- > > dadel1.png > > https://git.reviewboard.kde.org/media/uploaded/files/2016/04/28/0fc42425-947c-479e-9759-09c7a703a456__dadel1.png > Gwenview in fullscreen flips colors > > https://git.reviewboard.kde.org/media/uploaded/files/2016/05/04/1c0cab70-a54f-4533-b5e9-0b3af07ae087__dadel2.png > > > Thanks, > > Marco Martin > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
