Re: Review Request: Startup Feedback KWin Effect

2010-10-06 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/5529/ --- (Updated 2010-10-06 08:44:46.985052) Review request for kwin, Plasma and

Re: Review Request: Startup Feedback KWin Effect

2010-10-06 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/5529/#review7992 --- for the plasma part i'm ok. would be possible to use this effect

Re: Review Request: Startup Feedback KWin Effect

2010-10-06 Thread Martin Gräßlin
On 2010-10-06 11:08:18, Marco Martin wrote: for the plasma part i'm ok. would be possible to use this effect also when starting applications in other ways? (start menu, dolphin, whatever) It's used everywhere where you currently see the bouncing cursor. It's just that the startup

Re: Review Request: frame plasmoid: allow to add (separate) picture files to slideshow

2010-10-06 Thread michal seben
--- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/4256/ --- (Updated 2010-10-06 12:19:36.340865) Review request for Plasma. Summary

Re: Review Request: frame plasmoid: allow to add (separate) picture files to slideshow

2010-10-06 Thread michal seben
On 2010-09-24 10:33:40, Beat Wolf wrote: freeze is over. what is the status of this patch? Hi, patch isn't still accepted, it waits for a review ... - michal --- This is an automatically generated e-mail. To reply, visit:

Re: Review Request: frame plasmoid: allow to add (separate) picture files to slideshow

2010-10-06 Thread Anne-Marie Mahfouf
--- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/4256/#review7995 --- What's the use case for such an addition? The case that does

Re: Review Request: frame plasmoid: allow to add (separate) picture files to slideshow

2010-10-06 Thread michal seben
On 2010-10-06 12:45:50, Anne-Marie Mahfouf wrote: What's the use case for such an addition? The case that does not work in your patch is when you add 1 picture which is the same as a static picture but then you get the slideshow arrows. I am not very keen of adding code for new

Re: Review Request: Use Plasma::ScrollWidget in the widget explorer

2010-10-06 Thread Anselmo L. S. Melo
On 2010-10-01 17:21:28, Aaron Seigo wrote: update to current trunk if needed, and let's get this in as well. Anselmo L. S. Melo wrote: @agateau: I've updated this patch to current trunk [1] - I updated it when I wrote another patch [2], to check if both patches were ok together.

Re: Review Request: Use Plasma::ScrollWidget in the widget explorer

2010-10-06 Thread Anselmo L. S. Melo
On 2010-10-01 17:21:28, Aaron Seigo wrote: update to current trunk if needed, and let's get this in as well. Anselmo L. S. Melo wrote: @agateau: I've updated this patch to current trunk [1] - I updated it when I wrote another patch [2], to check if both patches were ok together.

Re: activitymanager api

2010-10-06 Thread Ivan Čukić
On 6 October 2010 21:40, Chani chan...@gmail.com wrote: summary for people who are not ivan: we were talking off-list about needed additions to activitymanager kded, to support the shiny fun session stuff. the new API that's already agreed on (modulo naming conventions) is:  void

Re: activitymanager api

2010-10-06 Thread Ivan Čukić
p.s. It will be no longer a kded module ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: activitymanager api

2010-10-06 Thread Ivan Čukić
neh, it'd be awkward api. activityClosedOrCancelled()? yuck. Ok no, it's quite literally impossible. ksmserver won't do it. unless you want to implement a queue for activities to sit in and wait for ksmserver to handle them... but that'd be overcomplicating things imho. It is needed for the