https://bugs.kde.org/show_bug.cgi?id=509711

--- Comment #2 from [email protected] ---
(In reply to David Redondo from comment #1)
> Is it a real problem? C will shown in the next round

No, it's a very minor issue, but I think that this still deserves a fix, if
it's trivial to fix.

Skipping an item in the queue means that the queue size is reduced by one, so
if a user expects their wallpaper slideshow to repeat in strict cycles, even
though the order of items is randomized, then this cycle shifts over time,
which can be annoying. (Yes, I do actually notice this from time to time,
otherwise I wouldn't have posted this)

So, as said, if it's trivial to fix by swapping items in the `SlideFilterModel
: QSortFilterProxyModel` instead of increasing the index by one, or maybe just
reversing the random order, then there's no reason not to do that. If I was
more familiar with the code, I'd submit a MR, but unfortunately, I am not.

Thanks for your consideration.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to