ngraham added a comment.
In D18809#427602 <https://phabricator.kde.org/D18809#427602>, @filipf wrote: > First and foremost I thank you as a user for giving this bit of code much needed love. > > If you'll have more interest in the slideshow in the future, implementing this might also be useful: https://bugs.kde.org/show_bug.cgi?id=186181 FWIW the Image Frame widget allows toggling between random and alphabetical ordering for its slideshow, so that might be useful as a template. For that matter, it might make sense to use this view in the Image Frame widget directly--after porting that feature over of course. Less code duplication for two views that are doing the same thing. REPOSITORY R120 Plasma Workspace BRANCH arcpatch-D18809_2 REVISION DETAIL https://phabricator.kde.org/D18809 To: davidre, #vdg, ngraham, davidedmundson Cc: filipf, mart, alexde, davidedmundson, ngraham, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
