Bug 261725

Summary: activitybar widget icons missing on startup
Product: [Plasma] plasma4 Reporter: Hans-Rudi Denzler <hr.denzler>
Component: activitiesAssignee: Plasma Bugs List <plasma-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: chanika, mursusoft
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: openSUSE   
OS: Linux   
Latest Commit: Version Fixed In:
Attachments: activity symbols after plasma-desktop restart

Description Hans-Rudi Denzler 2010-12-31 15:17:51 UTC
Version:           unspecified (using KDE 4.5.90) 
OS:                Linux

> kbuildsycoca4 && kquitapp plasma-desktop && kstart plasma-desktop
Now they appear.

Reproducible: Didn't try
Comment 1 Hans-Rudi Denzler 2010-12-31 15:21:56 UTC
Created attachment 55413 [details]
activity symbols after plasma-desktop restart
Comment 2 Chani 2010-12-31 15:44:04 UTC
by symbols, I assume you mean icons? :)
yes, my fault; it's a race condition in the dataengine. if it gets started before nepomuk comes up, you don't get icons.
what we need is some kind of signal when nepomuk does come up; it'd be possible to hack this in with a fake icon change if I need to avoid api changes, but, what would the Right Way be? have kactivitymanagerd emit a dbus signal when nepomuk becomes available and have KActivity* watch for that? or have KActivity* watch for nepomuk to appear?
hmm...
Comment 3 mursusoft 2011-01-04 15:35:20 UTC
Hope something can be done since icons are neat visual indicators :)
Comment 4 mursusoft 2011-01-04 15:37:31 UTC
They really are :D
Comment 5 Chani 2011-01-04 16:00:01 UTC
:) yes, I was working on it today; we've got a fix that works, the only rmaining question is what the most proper way to do it is.

unfortunately today's the tagging freeze for rc2, so the bugfix has to wait for tomorrow and you won't see it until the actual 4.6 release.
Comment 6 Chani 2011-01-05 15:28:31 UTC
SVN commit 1212025 by chani:

re-check hte icon when nepomuk comes up.
BUG: 261725

note: this isn't in trunk yet, me and ivan are discussing the Right way
to do it there.


 M  +9 -0      kactivityinfo.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1212025
Comment 7 Chani 2011-01-12 19:27:27 UTC
SVN commit 1214051 by chani:

revert r1212025: it's done better in r1214049.

CCBUG: 261725
oh, and it's fixed in trunk too; I forgot to cc that commit :)



 M  +0 -9      kactivityinfo.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1214051