davidre updated this revision to Diff 61115. davidre added a comment.
I couldm't crash it by clicking apply but noticed some type errors because the proxymodel din't have all the functionality of the underlying model. REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D22121?vs=61081&id=61115 BRANCH slideshow (branched from master) REVISION DETAIL https://phabricator.kde.org/D22121 AFFECTED FILES wallpapers/image/CMakeLists.txt wallpapers/image/backgroundlistmodel.h wallpapers/image/image.cpp wallpapers/image/image.h wallpapers/image/imagepackage/contents/ui/WallpaperDelegate.qml wallpapers/image/imagepackage/contents/ui/config.qml wallpapers/image/imagepackage/contents/ui/main.qml wallpapers/image/slidefiltermodel.cpp wallpapers/image/slidefiltermodel.h wallpapers/image/slidemodel.cpp wallpapers/image/slidemodel.h wallpapers/image/slideshowpackage/contents/config/main.xml To: davidre, #plasma Cc: msdobrescu, ngraham, filipf, plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart