clearing out review board =)

2008-10-28 Thread Aaron J. Seigo
hi all hell from my stupidly busy life at the moment. i see that there are some patches on review board. i'm hoping you guys can clear them out without me. which is to say, don't wait for me on those ones as i'll be stupid busy for the next few days =) -- Aaron J. Seigo humru othro a kohnu se

Re: plasma apps and screen information

2008-10-28 Thread Aaron J. Seigo
On Tuesday 28 October 2008, Guillaume Pothier wrote: > 2008/10/29 Aaron J. Seigo <[EMAIL PROTECTED]>: > > On Tuesday 28 October 2008, Guillaume Pothier wrote: > >> to have the corona register itself when created. Ok to commit? > > > > getting closer =) > > Cool :-) > > > first, the patch needs to g

Re: Review Request: Make sure that focused result item is visible when navigating with Tab key

2008-10-28 Thread Aaron Seigo
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.vidsolbach.de/r/245/#review237 --- Ship it! - Aaron On 2008-10-26 16:13:07, Dmitry Suzdalev wrote:

Re: plasma apps and screen information

2008-10-28 Thread Guillaume Pothier
2008/10/29 Aaron J. Seigo <[EMAIL PROTECTED]>: > On Tuesday 28 October 2008, Guillaume Pothier wrote: >> to have the corona register itself when created. Ok to commit? > > getting closer =) Cool :-) > > first, the patch needs to guard against containment() and/or corona() being 0. Do we need to

Re: plasma apps and screen information

2008-10-28 Thread Aaron J. Seigo
On Tuesday 28 October 2008, Guillaume Pothier wrote: > to have the corona register itself when created. Ok to commit? getting closer =) first, the patch needs to guard against containment() and/or corona() being 0. doing dynamic_casts like this: menu.exec(dynamic_cast(scene())->popupPosition(th

Re: GHNS & plasmapkg

2008-10-28 Thread Aaron J. Seigo
On Tuesday 28 October 2008, Petri Damstén wrote: > Run kbuildsyscoca4 from plasmapkg patch attached. this could probably be done via dbus and kded (faster, more reliable?) qdbus org.kde.kded /kbuildsycoca recreate so something like in the code: QDBusInterface dbus("org.kde.kded", "/kbui

Re: QuickAccess Plasmoid

2008-10-28 Thread Aaron J. Seigo
On Tuesday 28 October 2008, Loïc Marteau wrote: > http://www.kde-look.org/content/show.php?content=84128 > > I think than this one is really a killer one and should me moved ignoring the various ways it ignores plasma conventions in the code, isn't this simply what folderview should be in a horiz

Re: API Review: Tracking down the last beasties

2008-10-28 Thread Dan Meltzer
2008/10/28 Kevin Ottens <[EMAIL PROTECTED]>: > Le Tuesday 28 October 2008, Marco Martin a écrit : >> seriously tough, the only problem i perhaps see with webview is that it >> would suggest that as the other widgets is a proxywidget of QWebView, that >> is not since it uses directly QWebPage, so na

Re: API Review: Tracking down the last beasties

2008-10-28 Thread Kevin Ottens
Le Tuesday 28 October 2008, Marco Martin a écrit : > seriously tough, the only problem i perhaps see with webview is that it > would suggest that as the other widgets is a proxywidget of QWebView, that > is not since it uses directly QWebPage, so nativeWidget() will never return > anything meaningf

QuickAccess Plasmoid

2008-10-28 Thread Loïc Marteau
Hey all ! Sorry to not be able to work on plasma like I would like in this times. I'm really busy at work and hope than i can come back to send some little patch for review soon :). I would like to ask you if you know the quick Access plasmoid available on kde-look : http://www.kde-look.org/c

Re: pixmap leak in plasma?

2008-10-28 Thread Alex Merry
On Tuesday 28 October 2008 01:24:16 Thomas Fjellstrom wrote: > On Saturday 25 October 2008, you wrote: > > click on the toolbox button on the right (the "cashew") and then you can > > right click on the taskbar to get at its settings and what not. > > Ah.. I hope theres a more convenient way at som

GHNS & plasmapkg

2008-10-28 Thread Petri Damstén
Hi, I'm testing GHNS for comics and have some problems. knsrc has: InstallationCommand=plasmapkg -t comic -i %f GHNS loaded plasma packages should be installed with plasmapkg (like in kdebase/workspace/libs/plasma/scripting/plasmoids.knsrc)? If yes then: - Uninstall only removes the downloaded

How can I copy the NETRootInfo instance of a running plasma?

2008-10-28 Thread vatbier
Is it possible to read p->showing_desktop of NETRootInfo of plasma with a program I write myself? With NETRootinfo::showingDesktop() I should be able to do so but if I define a NetRootinfo info(QX11Info::display(), 0) the constructor for NETRootinfo sets p->showing_desktop to false. http://www.

Re: What is dbus command to toggle showdesktop?

2008-10-28 Thread vatbier
--- On Sun, 10/26/08, Aaron J. Seigo <[EMAIL PROTECTED]> wrote: > yes, you already figured out that there is no existing > command. it could be added to the show desktop applet, < though, which is in kdeplasma-addons. eh? Do you mean it can be added to dbus interface of plasma? Because I don't wa

Re: plasma apps and screen information

2008-10-28 Thread Guillaume Pothier
Ok, here comes a new version of the patch. I think I took into account all your comments: the new methods in Corona have a default implementation, I removed the ViewerCorona class, moved popupPosition to Corona, removed the convenience methods from View and Applet. The only remaining problem, I thi

Re: API Review: Tracking down the last beasties

2008-10-28 Thread Marco Martin
On Tuesday 28 October 2008, Dan Meltzer wrote: > 2008/10/28 Aaron J. Seigo <[EMAIL PROTECTED]>: > > On Monday 27 October 2008, Kevin Ottens wrote: > >> Hello all, > > > > i'm fine with all those changes. the most dubious imho is WebContent -> > > WebView, which must mean it's pretty good. i do pref

Re: API Review: Tracking down the last beasties

2008-10-28 Thread Dan Meltzer
2008/10/28 Aaron J. Seigo <[EMAIL PROTECTED]>: > On Monday 27 October 2008, Kevin Ottens wrote: >> Hello all, > > i'm fine with all those changes. the most dubious imho is WebContent -> > WebView, which must mean it's pretty good. i do prefer WebContent since > that's, well, what it is. it would be

Re: API Review: Tracking down the last beasties

2008-10-28 Thread Sebastian Kügler
On Tuesday 28 October 2008 08:04:58 Kevin Ottens wrote: > Le Tuesday 28 October 2008, Aaron J. Seigo a écrit : > > the most dubious imho is WebContent -> > > WebView, which must mean it's pretty good. i do prefer WebContent since > > that's, well, what it is. it would be like calling Plasma::Labe

Re: crash in Plasma::Meter

2008-10-28 Thread Sebastian Kügler
On Tuesday 28 October 2008 09:38:49 Marco Martin wrote: > On Tuesday 28 October 2008, Dan Meltzer wrote: > > 2008/10/28 Sebastian Kügler <[EMAIL PROTECTED]>: > > > I'm running into a crash, apparently in Plasma::Meter. It happens when > > > I initialize it. > > > > > > I've tried checking for a non

RE: tasks painting semplification

2008-10-28 Thread christian mollekopf
> From: [EMAIL PROTECTED]> To: plasma-devel@kde.org> Subject: Re: tasks > painting semplification> Date: Mon, 27 Oct 2008 19:23:17 +0100> > On Monday > 27 October 2008, Chani wrote:> > > however still speaking of the taskbar but > on a totally different area:> > > there are some issues i don't

Re: crash in Plasma::Meter

2008-10-28 Thread Marco Martin
On Tuesday 28 October 2008, Dan Meltzer wrote: > 2008/10/28 Sebastian Kügler <[EMAIL PROTECTED]>: > > I'm running into a crash, apparently in Plasma::Meter. It happens when I > > initialize it. > > > > I've tried checking for a non empty (valid, not null) size in > > Plasma::Meter, but that didn't

Re: Replacing Xesam runner with Nepomuk runner

2008-10-28 Thread Sebastian Trüg
On Monday 27 October 2008 23:52:32 Aaron J. Seigo wrote: > On Monday 27 October 2008, Sebastian Trüg wrote: > > Kevin (who wrote the Xesam runner) agrees with me on this. Now the only > > trouble is that I forgot to add the runner to the 4.2 feature list. I > > hope > > it's a bug fix: it doesn't w

Re: [PATCH-API-DOCU] AbstractRunner documentation

2008-10-28 Thread Sebastian Trüg
On Tuesday 28 October 2008 00:01:23 Aaron J. Seigo wrote: > i haven't read through your added docu (i trust you on that =), but did > take a look at the questions and here are some answers: > > +        // trueg: why is this method not protected? > > because it can be called by other classes if the

Re: API Review: Tracking down the last beasties

2008-10-28 Thread Kevin Ottens
Le Tuesday 28 October 2008, Aaron J. Seigo a écrit : > the most dubious imho is WebContent -> > WebView, which must mean it's pretty good. i do prefer WebContent since > that's, well, what it is. it would be like calling Plasma::Label a > Plasma::TextView ;) Ah right forgot this one, Plasma::Label