Bug 484307

Summary: Deleted wallpapers should be moved to the trash, not deleted instantly after clicking Apply
Product: [Plasma] plasmashell Reporter: piedro <piedro.kulman>
Component: Image & Slideshow wallpaper pluginsAssignee: Plasma Bugs List <plasma-bugs-null>
Status: RESOLVED FIXED    
Severity: wishlist CC: nate, niccolo, notmart, qydwhotmail
Priority: NOR Keywords: usability
Version First Reported In: 6.0.3   
Target Milestone: 1.0   
Platform: Other   
OS: Linux   
Latest Commit: https://invent.kde.org/plasma/plasma-workspace/-/commit/e5381dcdd8d7f44da63f2b10b910c3edf4b31b2b Version Fixed/Implemented In: 6.5.1
Sentry Crash Report:

Description piedro 2024-03-23 09:47:15 UTC 
SUMMARY
***
NOTE: If you are reporting a crash, please try to attach a backtrace with debug symbols.
See https://community.kde.org/Guidelines_and_HOWTOs/Debugging/How_to_create_useful_crash_reports
***


STEPS TO REPRODUCE
1. open desktop edit mode
2. In the dialog for choosing a wallpaper - delete one
3. Try to recover the wallpaper from the trashbin

OBSERVED RESULT
Wallpaper is not in the trashbin...  

EXPECTED RESULT
Deleted wallpapers (e.g. by accident) should be moved to the desktop trashbin

Operating System: openSUSE Tumbleweed 20240321
KDE Plasma Version: 6.0.2
KDE Frameworks Version: 6.0.0
Qt Version: 6.6.2
Kernel Version: 6.8.1-1-default (64-bit)
Graphics Platform: X11
Processors: 12 × AMD Ryzen 5 7600 6-Core Processor
Memory: 30.5 GiB of RAM
Graphics Processor: AMD Radeon RX 6700 XT
Manufacturer: ASUS

ADDITIONAL INFORMATION

Also while downloading new wallpapers there is no way to rename the image file.  

But many wallpapers are just named "02.Jpg" for example and without the ability to rename them during the wallpaper download process they overwrite each other or the installation of the wallpaper has to be omitted... 

Also even wallpapers with individual file names often aren't very descriptive. And since the naming is up to the author the option to rename the wallpaper image file while downloading would be very helpful. 

thx, p.
Comment 1 Niccolò Venerandi 2025-10-26 16:37:38 UTC 
Is this about wallpapers downloaded from the store?
Comment 2 Nate Graham 2025-10-26 17:29:23 UTC 
Actually I think there may be some confusion here. The trashcan icon makes it seem like the wallpaper will actually be deleted, but this isn't the case; it's simply removed from the view. That's why it doesn't appear in the trashcan. We should use a different icon for the button so this isn't misleading. I'll fix that.
Comment 3 Bug Janitor Service 2025-10-26 17:34:20 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/5942
Comment 4 Nate Graham 2025-10-26 17:51:02 UTC 
Git commit ee670adbecd6fa880e5983583e71af4d268d307c by Nate Graham.
Committed on 26/10/2025 at 17:50.
Pushed by ngraham into branch 'master'.

wallpapers/image: use more appropriate "remove wallpaper" icon

Right now it's a trashcan, which communicates that the wallpaper will be
deleted. This isn't accurate; use an X instead, which is what the HIG
recommends for this situation.
FIXED-IN: 6.5.1

M  +1    -1    wallpapers/image/imagepackage/contents/ui/WallpaperDelegate.qml

https://invent.kde.org/plasma/plasma-workspace/-/commit/ee670adbecd6fa880e5983583e71af4d268d307c
Comment 5 Nate Graham 2025-10-26 17:51:19 UTC 
Git commit e5381dcdd8d7f44da63f2b10b910c3edf4b31b2b by Nate Graham.
Committed on 26/10/2025 at 17:51.
Pushed by ngraham into branch 'Plasma/6.5'.

wallpapers/image: use more appropriate "remove wallpaper" icon

Right now it's a trashcan, which communicates that the wallpaper will be
deleted. This isn't accurate; use an X instead, which is what the HIG
recommends for this situation.
FIXED-IN: 6.5.1


(cherry picked from commit ee670adbecd6fa880e5983583e71af4d268d307c)

97f35a76 wallpapers/image: use more appropriate "remove wallpaper" icon

Co-authored-by: Nate Graham <nate@kde.org>

M  +1    -1    wallpapers/image/imagepackage/contents/ui/WallpaperDelegate.qml

https://invent.kde.org/plasma/plasma-workspace/-/commit/e5381dcdd8d7f44da63f2b10b910c3edf4b31b2b