Bug 472044

Summary:	File icon dropped on desktop gives busy indicator
Product:	[Plasma] plasmashell	Reporter:	Nicolas Fella <nicolas.fella>
Component:	general	Assignee:	Plasma Bugs List <plasma-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	kde, nate, notmart
Priority:	NOR	Keywords:	qt6
Version First Reported In:	master
Target Milestone:	1.0
Platform:	Other
OS:	Linux
See Also:	https://bugs.kde.org/show_bug.cgi?id=472161
Latest Commit:	https://invent.kde.org/plasma/plasma-workspace/-/commit/b6142b7b1079e04e6842843bef93e76347ab1790	Version Fixed/Implemented In:
Sentry Crash Report:

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