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

Tobias Fella <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |https://invent.kde.org/plas
                   |                            |ma/plasma-workspace/-/commi
                   |                            |t/39244fbe255eabc00c19d88ef
                   |                            |05e1dd9fb216878

--- Comment #7 from Tobias Fella <[email protected]> ---
Git commit 39244fbe255eabc00c19d88ef05e1dd9fb216878 by Tobias Fella, on behalf
of Fushan Wen.
Committed on 10/06/2026 at 09:24.
Pushed by tfella into branch 'Plasma/6.7'.

wallpapers/image: Fix slideshow not remembering the last shown picture

When a slideshow wallpaper is saved, the image URL is stored as a string
like "file:///path/to/image.jpg". When Plasma restarts, this URL string
is passed to SlideModel::indexOf().

The previous code used QUrl::fromLocalFile(packagePath) which incorrectly
treats the URL string as a local file path, creating a malformed URL like
"file:file:///path/to/image.jpg". This caused the image lookup to fail
(indexOf() returned -1), making the slideshow restart from the first image
instead of the last shown one.

Fix this by using QUrl::fromUserInput(packagePath) which correctly handles
both local file paths and URL strings.

Also add a unit test to verify indexOf() works correctly with URL strings.
Assisted-by: Kilo Code: glm-5
(cherry picked from commit e21f6d5c79ef018ff6bab3eafd1cb1491d47391e)

M  +21   -0    wallpapers/image/plugin/autotests/test_slidemodel.cpp
M  +1    -1    wallpapers/image/plugin/slidemodel.cpp

https://invent.kde.org/plasma/plasma-workspace/-/commit/39244fbe255eabc00c19d88ef05e1dd9fb216878

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

Reply via email to