Review Request: Override showOnlyCurrentDesktop for demandsAttention

[guitarist198 at yahoo.com]

> On 2008-08-13 07:19:52, Alex Merry wrote:
> > What happens when you click on a task that wants attention from another desktop?  Is the task brought to your current desktop, or does the desktop change to the one where the task is?
> > 
> > Also, it would be better if you didn't delete random empty lines in the code - they are often there to improve readability of the code, and it makes the patch harder to follow.

When a user clicks on a task that wants attention from another desktop, they are brought to that desktop and shown that application.  When they return to the last desktop, the task will remove itself since it no longer needs attention.

My apologies about the deletion of lines.  It may have happened after removing some debug outputs.


- distortedlogic


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


On 2008-08-10 10:09:39, distortedlogic wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.vidsolbach.de/r/131/
> -----------------------------------------------------------
> 
> (Updated 2008-08-10 10:09:39)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> This patch applies to the tasks.cpp and tasks.h files within the Tasks plasma applet.
> 
> It is in response to Bug 160613.  The patch allows a windowTaskItem that demands attention to override the showOnlyCurrentDesktop setting.  The user can then see that an application on a different desktop requires attention, click the window and be brought to the desktop.  When the window no longer requires attention it is no longer shown on other desktops.  This modification is relatively simple and only changes a few lines.  This is my first KDE patch and I would greatly appreciate the review.  I was unsure of the best way to bring forward this patch so I have submitted an attachment to the bugreport and sent in a request to the panel-devel mailinglist (Message is waiting on admin approval).  I do not mean to be too intrusive. 
> 
> I just recently discovered svn diff so the diff file found here is the most recent and most accurate compared to those found on the bugreport and mailing list. 
> 
> Thank you again. 
> 
> 
> This addresses bug 160613.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdebase/workspace/plasma/applets/tasks/tasks.h
>   /trunk/KDE/kdebase/workspace/plasma/applets/tasks/tasks.cpp
> 
> Diff: http://reviewboard.vidsolbach.de/r/131/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> distortedlogic
> 
>