Review Request: Unhide Panel when TaskManager's Task requests attention

[Aaron Seigo]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.vidsolbach.de/r/342/#review342
-----------------------------------------------------------


i don't like emitActivate either as it says what the code does rather than what functionality it provides.

requestAttention() might end up being confused for something that needs to be highlighted (as in the tasks widget for windows)? 

*thinks* 

needsVisualFocus()? pairs nicely with releaseVisualFocus, or is that too confusing?

- Aaron


On 2009-01-24 07:06:45, Diaa Sami wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.vidsolbach.de/r/342/
> -----------------------------------------------------------
> 
> (Updated 2009-01-24 07:06:45)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> Unhides Panel(s) containing a TaskManager when a task requests attention, this was done by emitting the Applet's activate() signal(and since AbstractTaskItem is not a friend, I had to make the public method emitActivate()).
> 
> I noticed that AbstractTaskItem::setTaskFlags(TaskWantsAttention) is called twice, is there a reason for this? it's not likely to cause problems but I'm just curious.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdebase/workspace/plasma/applets/tasks/abstracttaskitem.cpp
>   /trunk/KDE/kdelibs/plasma/applet.h
>   /trunk/KDE/kdelibs/plasma/applet.cpp
> 
> Diff: http://reviewboard.vidsolbach.de/r/342/diff
> 
> 
> Testing
> -------
> 
> basic testing only to make sure it works as intended
> 
> 
> Thanks,
> 
> Diaa
> 
>