[Icon Applet] Show busy indicator when launching app
ClosedPublic

Authored by broulik on Sun, Feb 3, 4:44 PM.

Details

Summary

Provides some feedback when clicking the icon.

Test Plan

Placed a Dolphin icon plasmoid in my panel, clicked it, got a busy indicator while it was launching.
Same on desktop. Same for some other apps.

It doesn't work for jump list actions because they are ran by commandline rather than actual desktop file.

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik created this revision.Sun, Feb 3, 4:44 PM
Restricted Application added a project: Plasma. · View Herald TranscriptSun, Feb 3, 4:44 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.Sun, Feb 3, 4:44 PM
broulik planned changes to this revision.Sun, Feb 3, 4:48 PM

Needs to set a timeout

broulik updated this revision to Diff 50819.Mon, Feb 4, 8:37 AM
  • Read timeout
ngraham accepted this revision.Mon, Feb 4, 2:06 PM
This revision is now accepted and ready to land.Mon, Feb 4, 2:06 PM
This revision was automatically updated to reflect the committed changes.
hein added a subscriber: hein.Mon, Feb 4, 3:43 PM

Hmm ... I guess it's OK, but using libtaskmanager would have been nicer maybe (centralized the timeout behavior, and the monitoring).