Review Request 110413: Show in the tasks applet the activities a task is available on

2013-05-13 Thread José Millán Soto

---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110413/
---

Review request for Plasma.


Description
---

This patch adds information about the activities on which a task is available 
to the tooltip.
Two methods to obtain information about the activities a task is available, 
activities and activityNames, where added to TaskManager::TaskItem.
Depending whether the applet is configured to show tasks which are not 
available in the current activity or not, the information displayed will be 
different.
If the applet is configured to show only tasks in current activity, only the 
tasks which are also available in other activities will have this information 
in the tooltip.


This addresses bug 307163.
http://bugs.kde.org/show_bug.cgi?id=307163


Diffs
-

  libs/taskmanager/taskitem.h 35606e2 
  libs/taskmanager/taskitem.cpp c9613c9 
  plasma/desktop/applets/tasks/windowtaskitem.cpp f840076 

Diff: http://git.reviewboard.kde.org/r/110413/diff/


Testing
---

In order to test this patch, three activities (called Activity 1, Activity 
2 and Activity 3) were created.
Various instances of KDialog were created, all of them in desktop 1, and they 
were assigned to various activities (The title of the dialog was set to AxDi, 
where x are the activities on which the dialog will be available).
All screenshots were taken on activity 1.
Screenshots 1 to 3 were taken with the taskbar configured to display tasks in 
all activities but only in desktop 1. Screenshots 4 to 6 were taken with the 
configuration to show tasks only in current activity. Screenshot 7 was taken 
with the taskbar displaying all tasks from all desktops. Screenshot 8 was 
created with the taskbar configured to group elements by program (activity 
information is not yet available).


File Attachments


Screenshot 1
  http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot1.png
Screenshot 2
  http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot2.png
Screenshot 3
  http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot3.png
Screenshot 4
  http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot4.png
Screenshot 5
  http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot5.png
Screenshot 6
  http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot6.png
Screenshot 7
  http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot7.png
Screenshot 8
  http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot8.png


Thanks,

José Millán Soto

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 110413: Show in the tasks applet the activities a task is available on

2013-05-13 Thread José Millán Soto

---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110413/
---

(Updated May 13, 2013, 11:10 a.m.)


Review request for Plasma.


Description
---

This patch adds information about the activities on which a task is available 
to the tooltip.
Two methods to obtain information about the activities a task is available, 
activities and activityNames, where added to TaskManager::TaskItem.
Depending whether the applet is configured to show tasks which are not 
available in the current activity or not, the information displayed will be 
different.
If the applet is configured to show only tasks in current activity, only the 
tasks which are also available in other activities will have this information 
in the tooltip.


This addresses bug 307163.
http://bugs.kde.org/show_bug.cgi?id=307163


Diffs
-

  libs/taskmanager/taskitem.h 35606e2 
  libs/taskmanager/taskitem.cpp c9613c9 
  plasma/desktop/applets/tasks/windowtaskitem.cpp f840076 

Diff: http://git.reviewboard.kde.org/r/110413/diff/


Testing
---

In order to test this patch, three activities (called Activity 1, Activity 
2 and Activity 3) were created.
Various instances of KDialog were created, all of them in desktop 1, and they 
were assigned to various activities (The title of the dialog was set to AxDi, 
where x are the activities on which the dialog will be available).
All screenshots were taken on activity 1.
Screenshots 1 to 3 were taken with the taskbar configured to display tasks in 
all activities but only in desktop 1. Screenshots 4 to 6 were taken with the 
configuration to show tasks only in current activity. Screenshot 7 was taken 
with the taskbar displaying all tasks from all desktops. Screenshot 8 was 
created with the taskbar configured to group elements by program (activity 
information is not yet available).


File Attachments


Screenshot 1
  http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot1.png
Screenshot 2
  http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot2.png
Screenshot 3
  http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot3.png
Screenshot 4
  http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot4.png
Screenshot 5
  http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot5.png
Screenshot 6
  http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot6.png
Screenshot 7
  http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot7.png
Screenshot 8
  http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot8.png


Thanks,

José Millán Soto

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 110413: Show in the tasks applet the activities a task is available on

2013-05-13 Thread Eike Hein

---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110413/#review32437
---


This isn't a call to reject this patch (which I think is a nice idea), but 
FWIW: The Task Manager widget is still intended to be replaced by a QML rewrite 
before feature freeze, and the QML bindings to Plasma's tooltip primitive 
currently don't allow for a lot in the way of layouting custom content, so I 
might not be able to recreate this too easily there, specifally having a line 
break between the two bits of info (unless the tooltip subtext prop accepts 
rich text). That said since I already had to fork the QML ToolTip component to 
get window preview thumbnails in there, I guess I could always fork my way 
further out of that dilemma.

In libplasma2 QML ToolTips are fortunately going to be much more flexible, 
ending the need for a fork.

- Eike Hein


On May 13, 2013, 11:10 a.m., José Millán Soto wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 http://git.reviewboard.kde.org/r/110413/
 ---
 
 (Updated May 13, 2013, 11:10 a.m.)
 
 
 Review request for Plasma.
 
 
 Description
 ---
 
 This patch adds information about the activities on which a task is available 
 to the tooltip.
 Two methods to obtain information about the activities a task is available, 
 activities and activityNames, where added to TaskManager::TaskItem.
 Depending whether the applet is configured to show tasks which are not 
 available in the current activity or not, the information displayed will be 
 different.
 If the applet is configured to show only tasks in current activity, only the 
 tasks which are also available in other activities will have this information 
 in the tooltip.
 
 
 This addresses bug 307163.
 http://bugs.kde.org/show_bug.cgi?id=307163
 
 
 Diffs
 -
 
   libs/taskmanager/taskitem.h 35606e2 
   libs/taskmanager/taskitem.cpp c9613c9 
   plasma/desktop/applets/tasks/windowtaskitem.cpp f840076 
 
 Diff: http://git.reviewboard.kde.org/r/110413/diff/
 
 
 Testing
 ---
 
 In order to test this patch, three activities (called Activity 1, Activity 
 2 and Activity 3) were created.
 Various instances of KDialog were created, all of them in desktop 1, and they 
 were assigned to various activities (The title of the dialog was set to AxDi, 
 where x are the activities on which the dialog will be available).
 All screenshots were taken on activity 1.
 Screenshots 1 to 3 were taken with the taskbar configured to display tasks in 
 all activities but only in desktop 1. Screenshots 4 to 6 were taken with the 
 configuration to show tasks only in current activity. Screenshot 7 was taken 
 with the taskbar displaying all tasks from all desktops. Screenshot 8 was 
 created with the taskbar configured to group elements by program (activity 
 information is not yet available).
 
 
 File Attachments
 
 
 Screenshot 1
   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot1.png
 Screenshot 2
   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot2.png
 Screenshot 3
   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot3.png
 Screenshot 4
   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot4.png
 Screenshot 5
   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot5.png
 Screenshot 6
   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot6.png
 Screenshot 7
   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot7.png
 Screenshot 8
   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot8.png
 
 
 Thanks,
 
 José Millán Soto
 


___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


if i define a function testfunc() in plasmoid /contents/main.js, when user add two instances of my plasmoid to taskbar, in these two instances, is function testfunc() a same object?

2013-05-13 Thread bruce
if i define testfunc.PROPERTY1=1, is it a same property?
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 110413: Show in the tasks applet the activities a task is available on

2013-05-13 Thread Eike Hein


 On May 13, 2013, 11:18 a.m., Eike Hein wrote:
  This isn't a call to reject this patch (which I think is a nice idea), but 
  FWIW: The Task Manager widget is still intended to be replaced by a QML 
  rewrite before feature freeze, and the QML bindings to Plasma's tooltip 
  primitive currently don't allow for a lot in the way of layouting custom 
  content, so I might not be able to recreate this too easily there, 
  specifally having a line break between the two bits of info (unless the 
  tooltip subtext prop accepts rich text). That said since I already had to 
  fork the QML ToolTip component to get window preview thumbnails in there, I 
  guess I could always fork my way further out of that dilemma.
  
  In libplasma2 QML ToolTips are fortunately going to be much more flexible, 
  ending the need for a fork.

Turns out that throwing a br/ in there is no problem, so this is easy to sync 
up with after all.


- Eike


---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110413/#review32437
---


On May 13, 2013, 11:10 a.m., José Millán Soto wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 http://git.reviewboard.kde.org/r/110413/
 ---
 
 (Updated May 13, 2013, 11:10 a.m.)
 
 
 Review request for Plasma.
 
 
 Description
 ---
 
 This patch adds information about the activities on which a task is available 
 to the tooltip.
 Two methods to obtain information about the activities a task is available, 
 activities and activityNames, where added to TaskManager::TaskItem.
 Depending whether the applet is configured to show tasks which are not 
 available in the current activity or not, the information displayed will be 
 different.
 If the applet is configured to show only tasks in current activity, only the 
 tasks which are also available in other activities will have this information 
 in the tooltip.
 
 
 This addresses bug 307163.
 http://bugs.kde.org/show_bug.cgi?id=307163
 
 
 Diffs
 -
 
   libs/taskmanager/taskitem.h 35606e2 
   libs/taskmanager/taskitem.cpp c9613c9 
   plasma/desktop/applets/tasks/windowtaskitem.cpp f840076 
 
 Diff: http://git.reviewboard.kde.org/r/110413/diff/
 
 
 Testing
 ---
 
 In order to test this patch, three activities (called Activity 1, Activity 
 2 and Activity 3) were created.
 Various instances of KDialog were created, all of them in desktop 1, and they 
 were assigned to various activities (The title of the dialog was set to AxDi, 
 where x are the activities on which the dialog will be available).
 All screenshots were taken on activity 1.
 Screenshots 1 to 3 were taken with the taskbar configured to display tasks in 
 all activities but only in desktop 1. Screenshots 4 to 6 were taken with the 
 configuration to show tasks only in current activity. Screenshot 7 was taken 
 with the taskbar displaying all tasks from all desktops. Screenshot 8 was 
 created with the taskbar configured to group elements by program (activity 
 information is not yet available).
 
 
 File Attachments
 
 
 Screenshot 1
   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot1.png
 Screenshot 2
   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot2.png
 Screenshot 3
   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot3.png
 Screenshot 4
   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot4.png
 Screenshot 5
   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot5.png
 Screenshot 6
   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot6.png
 Screenshot 7
   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot7.png
 Screenshot 8
   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot8.png
 
 
 Thanks,
 
 José Millán Soto
 


___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: if i define a function testfunc() in plasmoid /contents/main.js, when user add two instances of my plasmoid to taskbar, in these two instances, is function testfunc() a same object?

2013-05-13 Thread Giorgos Tsiapaliokas
Hello,

your function will be a function in all the instances
of your plasmoid, but it won't live in the same memory.

What exactly do you want to do?



On 13 May 2013 14:22, bruce bruce...@gmail.com wrote:

 if i define testfunc.PROPERTY1=1, is it a same property?
 ___
 Plasma-devel mailing list
 Plasma-devel@kde.org
 https://mail.kde.org/mailman/listinfo/plasma-devel




-- 
Giorgos Tsiapaliokas (terietor)

terietor.org
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 110413: Show in the tasks applet the activities a task is available on

2013-05-13 Thread José Millán Soto


 On May 13, 2013, 11:18 a.m., Eike Hein wrote:
  This isn't a call to reject this patch (which I think is a nice idea), but 
  FWIW: The Task Manager widget is still intended to be replaced by a QML 
  rewrite before feature freeze, and the QML bindings to Plasma's tooltip 
  primitive currently don't allow for a lot in the way of layouting custom 
  content, so I might not be able to recreate this too easily there, 
  specifally having a line break between the two bits of info (unless the 
  tooltip subtext prop accepts rich text). That said since I already had to 
  fork the QML ToolTip component to get window preview thumbnails in there, I 
  guess I could always fork my way further out of that dilemma.
  
  In libplasma2 QML ToolTips are fortunately going to be much more flexible, 
  ending the need for a fork.
 
 Eike Hein wrote:
 Turns out that throwing a br/ in there is no problem, so this is easy 
 to sync up with after all.

If that's the case, I suppose I should do a commit which includes the changes 
in TaskItem but not the changes in WindowTaskItem.
Am I right?


- José


---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110413/#review32437
---


On May 13, 2013, 11:10 a.m., José Millán Soto wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 http://git.reviewboard.kde.org/r/110413/
 ---
 
 (Updated May 13, 2013, 11:10 a.m.)
 
 
 Review request for Plasma.
 
 
 Description
 ---
 
 This patch adds information about the activities on which a task is available 
 to the tooltip.
 Two methods to obtain information about the activities a task is available, 
 activities and activityNames, where added to TaskManager::TaskItem.
 Depending whether the applet is configured to show tasks which are not 
 available in the current activity or not, the information displayed will be 
 different.
 If the applet is configured to show only tasks in current activity, only the 
 tasks which are also available in other activities will have this information 
 in the tooltip.
 
 
 This addresses bug 307163.
 http://bugs.kde.org/show_bug.cgi?id=307163
 
 
 Diffs
 -
 
   libs/taskmanager/taskitem.h 35606e2 
   libs/taskmanager/taskitem.cpp c9613c9 
   plasma/desktop/applets/tasks/windowtaskitem.cpp f840076 
 
 Diff: http://git.reviewboard.kde.org/r/110413/diff/
 
 
 Testing
 ---
 
 In order to test this patch, three activities (called Activity 1, Activity 
 2 and Activity 3) were created.
 Various instances of KDialog were created, all of them in desktop 1, and they 
 were assigned to various activities (The title of the dialog was set to AxDi, 
 where x are the activities on which the dialog will be available).
 All screenshots were taken on activity 1.
 Screenshots 1 to 3 were taken with the taskbar configured to display tasks in 
 all activities but only in desktop 1. Screenshots 4 to 6 were taken with the 
 configuration to show tasks only in current activity. Screenshot 7 was taken 
 with the taskbar displaying all tasks from all desktops. Screenshot 8 was 
 created with the taskbar configured to group elements by program (activity 
 information is not yet available).
 
 
 File Attachments
 
 
 Screenshot 1
   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot1.png
 Screenshot 2
   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot2.png
 Screenshot 3
   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot3.png
 Screenshot 4
   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot4.png
 Screenshot 5
   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot5.png
 Screenshot 6
   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot6.png
 Screenshot 7
   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot7.png
 Screenshot 8
   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot8.png
 
 
 Thanks,
 
 José Millán Soto
 


___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


kde-workspace and plasma2

2013-05-13 Thread Marco Martin
Hi all,
besides having plasma2 pieces in the plasma-workspace repository, there will 
be the need of components in kde-workspace as well.

there has been some work on there already:
there is:
* a qml2 desktop containment
* a qml desktop (and now panel too) toolboxes
* a kded that says what formfactor we are running on
in the end the plasma part of kde-workspace should look like
http://community.kde.org/Plasma/KdeWorkspaceRepoStructure
(at least so was discussed in nuremberg)

since i wanted to start on porting the good ol' image wallpaper (mostly to 
fuigure out what we need on workpace-side for doing all the wallpapers we 
need), i think is good for it to stay in kde-workpace.

I would propose to make the branch 
plasma/sebas/desktop-qml-plasma2
that is now hosting the desktop containment and the toolboxes to host the 
refactor of the filesystem as well (and there the only buildable part for now 
would be under plasma/ the part that have been ported)

then the other parts (like kwin, ksysguard etc) can work on ports in their own 
branches (or own repo, their choice) and merge when is ready (or stay on 
different repo)

features on plsma2 porting would be branches of this branch derived from 
plasma/sebas/desktop-qml-plasma2 and merged when the author of the task feels 
is ready to be tested by others.

If people agrees i can do right away this branch with a beginning of the 
porting of the filesystem structure to what's in the wiki page

Cheers,
Marco Martin
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: if i define a function testfunc() in plasmoid /contents/main.js, when user add two instances of my plasmoid to taskbar, in these two instances, is function testfunc() a same object?

2013-05-13 Thread bruce
在 2013年5月13日 星期一 15:57:16,Giorgos Tsiapaliokas 写道:


Hello,


your function will be a function in all the instances
of your plasmoid, but it won't live in the same memory.


What exactly do you want to do?




On 13 May 2013 14:22, bruce bruce...@gmail.com[1] wrote:


if i define testfunc.PROPERTY1=1, is it a same 
property?___Plasma-devel mailing 
list

Plasma-devel@kde.org[2]
https://mail.kde.org/mailman/listinfo/plasma-devel[3]





-- 


Giorgos Tsiapaliokas (terietor)


terietor.org[4]



my purpose is finding a variable or anything similar which exists in different 
instances of a 
plasmoid, similar with shared memory between processes, shared object between 
java 
threads, etc.




[1] mailto:bruce...@gmail.com
[2] mailto:Plasma-devel@kde.org
[3] https://mail.kde.org/mailman/listinfo/plasma-devel
[4] http://terietor.org
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


there seems to be no dataengines or datasources can provide harddisk's temperature, right?

2013-05-13 Thread bruce
so all i have to do is install hddtemp or smartctl?
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


UI refactor

2013-05-13 Thread Shantanu Tushar Jha
Hi Fabian,

I finally got some time to look at the refactored branch, and it does look
good :) Especially the keyboard working in music mode. Awesome work!
However, I guess we made quite a lot of changes to master and I am guessing
it will take some effort. Have you tried merging master into your branch?
If not, lets decide on a common time and get this done :)

-- 
Shantanu Tushar(UTC +0530)
http://www.shantanutushar.com
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: if i define a function testfunc() in plasmoid /contents/main.js, when user add two instances of my plasmoid to taskbar, in these two instances, is function testfunc() a same object?

2013-05-13 Thread Marco Martin
On Monday 13 May 2013, bruce wrote:
 
 if i define testfunc.PROPERTY1=1, is it a same
 property?___Plasma-devel
 mailing list
 
 Plasma-devel@kde.org[2]
 https://mail.kde.org/mailman/listinfo/plasma-devel[3]

 my purpose is finding a variable or anything similar which exists in
 different instances of a plasmoid, similar with shared memory between
 processes, shared object between java threads, etc.

plasmoids are supposed to be at least a bit sandboxed and not supposed to 
shareglobal statics or stuff like that.

on the other hand dataengines have usually only one instance for each, 
regardless how many plasmoids are using them

as usual what should be done is to step back and look exactly what you want to 
do, what your final result in terms of functionality should be, rather than 
starting from a possible solution

-- 
Marco Martin
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 110420: Bugfix for bug #311364

2013-05-13 Thread Romário Rios

---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110420/
---

(Updated May 14, 2013, 1:22 a.m.)


Review request for kde-workspace and Plasma.


Changes
---

Added kde-workspace group


Description
---

Fixes bug #311364. I'm not entirely sure if that's a bug or a feature, but 
since it's a simple one-liner, I figured I'd just drop the diff and let you 
decide.


This addresses bug 311364.
http://bugs.kde.org/show_bug.cgi?id=311364


Diffs
-

  plasma/generic/applets/notifications/contents/ui/main.qml e867517 

Diff: http://git.reviewboard.kde.org/r/110420/diff/


Testing
---

Popup goes away when it loses focus.


Thanks,

Romário Rios

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request 110420: Bugfix for bug #311364

2013-05-13 Thread Romário Rios

---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110420/
---

(Updated May 14, 2013, 1:56 a.m.)


Status
--

This change has been discarded.


Review request for kde-workspace and Plasma.


Description
---

Fixes bug #311364. I'm not entirely sure if that's a bug or a feature, but 
since it's a simple one-liner, I figured I'd just drop the diff and let you 
decide.


This addresses bug 311364.
http://bugs.kde.org/show_bug.cgi?id=311364


Diffs
-

  plasma/generic/applets/notifications/contents/ui/main.qml e867517 

Diff: http://git.reviewboard.kde.org/r/110420/diff/


Testing
---

Popup goes away when it loses focus.


Thanks,

Romário Rios

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel