Re: [Kde-pim] SoC Updates -- Kontact and Skrooge Plasma Dashboards

2010-04-23 Thread Ryan Rix
On Fri 23 April 2010 11:17:16 am Aaron J. Seigo wrote: > if it's in kdebase-workspace, then other apps can't use it (skrooge, > kontact). > > if it's in kdebase-runtime, you can't subclass it (runtime is not a valid > build time dep) As already discussed in IRC, I am personally 100% good with t

Re: Review Request: [Quicklaunch] Refactoring of the Quicklaunch applet

2010-04-23 Thread Lukas Appelhans
Am Freitag 23 April 2010 21:08:34 schrieb Ingomar Wesp: > --- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/3786/ > --- > > Review request

Review Request: [Quicklaunch] Refactoring of the Quicklaunch applet

2010-04-23 Thread Ingomar Wesp
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/3786/ --- Review request for Plasma. Summary --- This is my proposed patch for the re

Re: Review Request: Expose KStatusNotifierItem menus on DBus

2010-04-23 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/3787/#review5191 --- Ship it! is still a work in progress, but as soon the dependency from

Re: Review Request: Show KStatusNotifierItem menus exposed over DBus

2010-04-23 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/3788/#review5193 --- Ship it! as the other one, ship it, modulo the dbusmenu lib in kdesup

Re: Review Request: basic activity manager

2010-04-23 Thread Chani Armitage
> On 2010-04-23 08:20:21, Ivan Cukic wrote: > > First, I'm getting two compilation problems - AbstractIcon and > > AbstractIconList are not in Plasma namespace. > > > > The other thing is that AbstractIcon and AbstractIconList don't have > > PLASMAGENERICSHELL_EXPORT so the linker fails as wel

Re: Review Request: basic activity manager

2010-04-23 Thread Chani Armitage
> On 2010-04-23 16:33:26, Aaron Seigo wrote: > > /trunk/KDE/kdebase/workspace/plasma/desktop/shell/controllerwindow.cpp, > > lines 312-314 > > > > > > does the activity manager really need to care if we don't have a > >

Re: [Kde-pim] SoC Updates -- Kontact and Skrooge Plasma Dashboards

2010-04-23 Thread Aaron J. Seigo
On April 23, 2010, Marco Martin wrote: > if after some tests on a working thing it seems that the newspaper is > really what we need, i was thinking about making a more barebone > containment out of it and put it in workspace/libs (uhm, runtime perhaps?) > and subclass it with my custom toolbox and

Re: [Kde-pim] SoC Updates -- Kontact and Skrooge Plasma Dashboards

2010-04-23 Thread Marco Martin
On Friday 23 April 2010, Aaron J. Seigo wrote: > > if that results in the newspaper containment being "The One", then we'll > move that containment to kdebase/runtime/plasma/containments/. we already > have plasmapkg and the javascript Plasma::ScriptEngine in > kdebase/runtime/plasma for the same

Re: Plasma Desktop Scripting Tutorial for Distros

2010-04-23 Thread Aaron J. Seigo
hi :) On April 23, 2010, Jonathan Riddell wrote: > Aaron Seigo has offered to do a tutorial on Plasma Desktop Scripting. > He's proposing a session of around two hours on IRC to show desktop > scripting to distribution developers and anyone else who's interested. thanks for spearheading this. a

Re: [Kde-pim] SoC Updates -- Kontact and Skrooge Plasma Dashboards

2010-04-23 Thread Aaron J. Seigo
On April 22, 2010, Ryan Rix wrote: > * I'm currently reimplementing the WidgetBrowser in sounds good. > * Siddharth is currently working on eliminating our other dependencies, > namely the config dialogs. you shouldn't need to configure much, if anything, i'd think? what other deps are you faci

Re: Review Request: basic activity manager

2010-04-23 Thread Aaron Seigo
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/3780/#review5187 --- Ship it! obviously it's a work in progress and has its rough edges, b

Review Request: Show KStatusNotifierItem menus exposed over DBus

2010-04-23 Thread Aurélien Gâteau
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/3788/ --- Review request for Plasma. Summary --- ( This patch depends on http://revie

Review Request: Expose KStatusNotifierItem menus on DBus

2010-04-23 Thread Aurélien Gâteau
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/3787/ --- Review request for kdelibs and Plasma. Summary --- This patch makes KStatus

Re: Review Request: Added Plasma::PendulumCurve class

2010-04-23 Thread Aaron Seigo
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/3513/#review5185 --- Ship it! please commit these changes with two separate commits: one f

Re: Review Request: "easingCurve" property now belongs to Plasma::EasingAnimation class

2010-04-23 Thread Aaron Seigo
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/3783/#review5184 --- yes, because it's not binary compat with 4.4 we can't do this. the bes

Plasma Desktop Scripting Tutorial for Distros

2010-04-23 Thread Jonathan Riddell
Aaron Seigo has offered to do a tutorial on Plasma Desktop Scripting. He's proposing a session of around two hours on IRC to show desktop scripting to distribution developers and anyone else who's interested. If you're interested please indicate when you would be available next week on this Doodl

Re: Review Request: Added Plasma::PendulumCurve class

2010-04-23 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/3513/#review5182 --- Ship it! all nice now :) - Marco On 2010-04-22 22:26:21, Bruno Abi

Re: Review Request: Add simple Type-and-Select feature to Folderview applet

2010-04-23 Thread Shantanu Tushar Jha
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2659/ --- (Updated 2010-04-23 09:46:42.187201) Review request for Plasma, usability, Fredr

Re: Review Request: Add simple Type-and-Select feature to Folderview applet

2010-04-23 Thread Shantanu Tushar Jha
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2659/ --- (Updated 2010-04-23 09:42:14.966432) Review request for Plasma, usability, Fredr

Re: Review Request: basic activity manager

2010-04-23 Thread Ivan Cukic
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/3780/#review5180 --- First, I'm getting two compilation problems - AbstractIcon and Abstrac

Re: SoC Updates -- Kontact and Skrooge Plasma Dashboards

2010-04-23 Thread Ivan Čukić
> You might want more control over the applets than the newspaper > containment---again, I'm not sure, but in my experience, apps with > different use-cases, form factors, and intended plasma widgets will want While I agree that no containment can suit all applications, if we choose the way of i

Re: SoC Updates -- Kontact and Skrooge Plasma Dashboards

2010-04-23 Thread Leo Franchi
On 23 Apr 2010, at 07:32, Ryan Rix wrote: > > After that, we get to a really interesting situation: containments. > Currently all the contianments are in kdebase-workspace, where they > /probably/ should be. Of course, that means that we currently cannot > depend on any of them being there, inc