Bug 294388 - Show Activity icon in alt-f3 Activity menu
Summary: Show Activity icon in alt-f3 Activity menu
Status: RESOLVED FIXED
Alias: None
Product: kactivitymanagerd
Classification: Plasma
Component: general (show other bugs)
Version: 4.9.0
Platform: Unlisted Binaries Linux
: NOR wishlist
Target Milestone: ---
Assignee: Ivan Čukić
URL:
Keywords: junior-jobs
Depends on:
Blocks: extramile
  Show dependency treegraph
 
Reported: 2012-02-18 21:46 UTC by Will Stephenson
Modified: 2015-05-02 15:34 UTC (History)
6 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 Will Stephenson 2012-02-18 21:46:13 UTC
Seeing activities' icons would make it faster to choose an Activity from the menu.
Comment 1 Vincent de Phily 2012-05-31 11:32:09 UTC
Works for me (KDE 4.8.3).
Comment 2 Maarten De Meyer 2012-07-24 16:35:22 UTC
Woks for me too (KDE 4.9 RC2)
Comment 3 Janek Bevendorff 2012-07-24 18:06:41 UTC
Does only work for icons set by the user.
Default icons (the colored balls) are not shown. This is pretty ugly as it looks like something is broken.

Re-close if you don't agree, but I think the menu should either show ALL icons or NONE.
Comment 4 Thomas Lübking 2012-08-05 16:38:27 UTC
Short comments:
a) this is not a kwin issue, for it just relies on KActivities::Info::icon() for _all_ activities.
b) MORE IMPORTANT:
that and KActivities::Info::name() are STILL SYNC DBUS calls in usermanager.cpp / void Workspace::activityPopupAboutToShow()
:-\
Comment 5 Martin Flöser 2012-10-24 19:47:01 UTC
moving to KActivities, see comment #4
Comment 6 Ivan Čukić 2012-10-24 21:16:21 UTC
> this is not a kwin issue, for it just relies on KActivities::Info::icon() for _all_ activities.

Plasma just conveniently provides identicons for the activities that have no icons. So, no other user sees them. I don't really think this code (libplasma-dependant) should be in KActivities.

On the other hand, I do agree it is not really a kwin issue.

> that and KActivities::Info::name() are STILL SYNC DBUS calls in usermanager.cpp / 
> void Workspace::activityPopupAboutToShow()

This is a KWin implementation issue - the name() method wouldn't be sync if it were used in a long-lived Info class instance. When doing the 

{
   Info activity(id);
   activity.name();
   ...
}

it is not giving it the time to fetch the name from the service before it is requested. 

KActivities now provide a data model that lists activities. It should be used in kwin at some point.
Anyhow, since we survived with it for a few releases now I think we should wait for this change for after 4.10
Comment 7 Thomas Lübking 2012-10-24 21:46:01 UTC
(In reply to comment #6)

> This is a KWin implementation issue - the name() method wouldn't be sync if
> it were used in a long-lived Info class instance. When doing the 

The comment is older than the kactivity changes ;-P

And yes - the activity implementation in kwin is still rather a mess.
Needs a complete review & cleanup (too much code doubling from the virtual desktops, too many TODOs. Too much commented desktop cloning code)

I ever wanted to call on action for this. Any plans for this WE?
Comment 8 Ivan Čukić 2012-10-24 21:49:23 UTC
Ah, damn, the new mail was about Martin changing the component :D

About the plans - hmh, not sure how pleasant would be to unify the code between activities and VDs since they do have slight differences in kwin behaviour.
Comment 9 Thomas Lübking 2012-10-24 22:06:08 UTC
I didn't really mean to "unify" the code, but esp. to get the activty management in shape (ie, get rid of "//TODO do i need this" comments), see what's left and eventually merge some shared basics.

The next step would then be to figure whether VD and activities should actually remain the way they are or whether activities should become some sort of advanced VD (as they are now) - implementing an extended version of the NETWM spec and VD more like a viewport (with static desktop and floating windows) - but that would first have to pass kcd anyway.
Comment 10 Johannes Georgi 2014-01-10 14:52:18 UTC
Can't this be closed?
Works for me on 4.10.5
Comment 11 Vincent de Phily 2014-01-10 18:07:29 UTC
Still valid as far as I can see (kde 4.11.5) : user-chosen icons are visible, but not software-generated patterns.

If this is an issue of the software-generated icons being implemented by plasma but not kwin nor kactivities, it seems to me that the fix is to move plasma's implementation over to KActivities::Info::icon() ? Otherwise users will keep getting confused with activity icons showing in one place but not another.
Comment 12 Johannes Georgi 2014-01-10 23:36:07 UTC
Can you please look at the image and tell me if you see this bug?
http://i42.tinypic.com/2i0t402.jpg
I do not mean to get this closed, I just would like to know if I am missing something.
Comment 13 Thomas Lübking 2014-01-10 23:37:21 UTC
The bug is about the popup menu entry (rightlick a window titlebar)
Comment 14 Johannes Georgi 2014-01-10 23:42:07 UTC
Okay, I see it too.
Comment 15 Ivan Čukić 2015-05-02 15:34:16 UTC
'Coloured balls' do not exist any more. Closing this.