https://bugs.kde.org/show_bug.cgi?id=509711
--- Comment #5 from [email protected] --- (In reply to Nate Graham from comment #4) > The code for random wallpaper slideshows is already very complex; > surprisingly so, even. I think adding additional complexity for something > quite minor is unfortunately not worth the risk. Thanks for the idea anyway, > though! Would you still accept a merge request which adds a very simple `SlideFilterModel::reverse()` for reversing the `QList<int> m_randomOrder` instead of the `m_currentSlide += 1` increment? I don't think this is very complex at all. $ git diff --shortstat wallpapers/ 3 files changed, 9 insertions(+), 1 deletion(-) I tested this with added debug log messages and it's working fine. If you don't want a MR for this, then I can keep this patch in my own plasma-workspace builds, but I don't see a reason not to upstream this. -- You are receiving this mail because: You are watching all bug changes.
