----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127091/#review92470 -----------------------------------------------------------
Is this testable at all? - Albert Astals Cid On Feb. 16, 2016, 4:52 p.m., Daniel Vrátil wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/127091/ > ----------------------------------------------------------- > > (Updated Feb. 16, 2016, 4:52 p.m.) > > > Review request for KDE Frameworks, Plasma and David Edmundson. > > > Bugs: 359388 > http://bugs.kde.org/show_bug.cgi?id=359388 > > > Repository: plasma-framework > > > Description > ------- > > Fix sni-qt icons (or generally any non-theme icons) not showing in systray > after 5184ac94. The fallback code was assuming that the name always comes > from QIcon::iconTheme icon. > > > Diffs > ----- > > src/declarativeimports/core/iconitem.cpp 1d7921a > > Diff: https://git.reviewboard.kde.org/r/127091/diff/ > > > Testing > ------- > > I can see sni-qt apps icons in Plasma systray again. > > > Thanks, > > Daniel Vrátil > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
