----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125134/#review85573 -----------------------------------------------------------
components/shellprivate/widgetexplorer/plasmaappletitemmodel_p.h (line 55) <https://git.reviewboard.kde.org/r/125134/#comment59140> are you overriding a method here? If yes please add override. If no I'm suggesting to not use the same name as the virtual method. - Martin Gräßlin On Sept. 10, 2015, 1:36 p.m., Marco Martin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125134/ > ----------------------------------------------------------- > > (Updated Sept. 10, 2015, 1:36 p.m.) > > > Review request for Plasma and andreas kainz. > > > Repository: plasma-workspace > > > Description > ------- > > As suggested by the VDG, add the possibility to have screenshots for > plasmoids. > the widget explorer will be modified as well to show the screenshots in the > applet list when available > > > Diffs > ----- > > components/shellprivate/widgetexplorer/plasmaappletitemmodel.cpp 99e9add > components/shellprivate/widgetexplorer/plasmaappletitemmodel_p.h f9cc89f > > Diff: https://git.reviewboard.kde.org/r/125134/diff/ > > > Testing > ------- > > > Thanks, > > Marco Martin > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel