Review Request: Override showOnlyCurrentDesktop for demandsAttention

guitarist198 at yahoo.com guitarist198 at yahoo.com
Tue Aug 19 03:33:42 CEST 2008


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

(Updated 2008-08-18 18:33:42.206447)


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



More information about the Plasma-devel mailing list