Bug 472044 - File icon dropped on desktop gives busy indicator
Summary: File icon dropped on desktop gives busy indicator
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: general (show other bugs)
Version: master
Platform: Other Linux
: NOR normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords: qt6
Depends on:
Blocks:
 
Reported: 2023-07-07 12:23 UTC by Nicolas Fella
Modified: 2023-08-06 09:50 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Nicolas Fella 2023-07-07 12:23:07 UTC
STEPS TO REPRODUCE
1. Drag a file in Dolphin 
2. Drop onto folder view
3. In drop menu choose "Add icon"

OBSERVED RESULT
Created icon applet has busy indicator that keeps spinning

EXPECTED RESULT
Busy indicator stops eventually

SOFTWARE/OS VERSIONS
KDE Plasma Version: master
KDE Frameworks Version: master
Qt Version: 6.5.1

ADDITIONAL INFORMATION
After plasmashell --replace there's no busy indicator any more
Comment 1 Nicolas Fella 2023-07-08 23:19:37 UTC
The call to setBusy(false) in https://invent.kde.org/plasma/plasma-workspace/-/blob/master/applets/icon/iconapplet.cpp#L271 is not working
Comment 2 Nate Graham 2023-07-11 16:37:02 UTC
Can confirm. While testing, I also found Bug 472161.
Comment 3 Bug Janitor Service 2023-08-05 13:20:04 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/3156
Comment 4 Kai Uwe Broulik 2023-08-06 09:50:38 UTC
Git commit b6142b7b1079e04e6842843bef93e76347ab1790 by Kai Uwe Broulik.
Committed on 06/08/2023 at 11:36.
Pushed by broulik into branch 'master'.

BasicAppletContainer: Use correct "busy" reference

It's on applet.plasmoid like the rest of the applet properties.

M  +1    -1    components/containmentlayoutmanager/qml/BasicAppletContainer.qml

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