Re: API Review: Tracking down the last beasties

2008-10-27 Thread Aaron J. Seigo
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 like calling Plasma::Label a Plasma::TextView ;) any

Re: crash in Plasma::Meter

2008-10-27 Thread Dan Meltzer
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 seem to help. > > Can somebody make sense of it? It looks

crash in Plasma::Meter

2008-10-27 Thread Sebastian Kügler
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 seem to help. Can somebody make sense of it? -- sebas http://www.kde.org | http://vizZzion.org | GPG Key ID: 9

Re: pixmap leak in plasma?

2008-10-27 Thread Thomas Fjellstrom
On Saturday 25 October 2008, you wrote: > On Saturday 25 October 2008, Thomas Fjellstrom wrote: > > On Friday 24 October 2008, you wrote: > > > On Friday 24 October 2008, Thomas Fjellstrom wrote: > > > > nearly 300MB in pixmaps. I'm using trunk as of a day or two ago. > > > > > > things to try: > >

API Review: Tracking down the last beasties

2008-10-27 Thread Kevin Ottens
Hello all, I've been taking some time to complete the API reviews, we only have a few smallish issues left. Since Aaron basically said he'd not be really around this week[*], but still we've to move on we'll proceed in the following way: - This mail contains the list of changes I'm planning; -

Re: plasma apps and screen information

2008-10-27 Thread Aaron J. Seigo
On Monday 27 October 2008, Guillaume Pothier wrote: > 2008/10/27 Aaron J. Seigo <[EMAIL PROTECTED]>: > >> The latter should probably return a QRegion instead, according to what > >> you said about toolbox placement. > > > > yes, it should be a QRegion. > > ok, I'll use QRegion in the API, but for n

Re: [PATCH-API-DOCU] AbstractRunner documentation

2008-10-27 Thread Aaron J. Seigo
On Monday 27 October 2008, Sebastian Trüg wrote: > Hi guys, > > while implementing the Nepomuk Search runner I stumbled over the > incomplete, partially outdated, and partially wrong documentation of > Plasma::AbstractRunner. I tried to fix it as far as I could from looking at > the code. Please re

Re: TaskManager::windowChangedGeometry() displacable?

2008-10-27 Thread Aaron J. Seigo
On Monday 27 October 2008, christian mollekopf wrote: > I noticed in > void TaskManager::windowChanged(WId w, unsigned int dirty) > the windowChangedGeometry(t) signal isn't neccessary anymore and we can > easely switch to the windowChanged(changes) signal. > > Should i change this or is this on p

Re: Replacing Xesam runner with Nepomuk runner

2008-10-27 Thread Aaron J. Seigo
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 work now, but it will. put it in. more good work from the nepomukers =

Re: tasks painting semplification

2008-10-27 Thread Aaron J. Seigo
On Monday 27 October 2008, Jamboarder wrote: > > From: Marco Martin <[EMAIL PROTECTED]> > > aaanyways, the painting function it's really a mess because when i added > > the support for svg theming i decided to add a fallback for every > > element, so when it does not exist in the svg just paints a

not overly available this week

2008-10-27 Thread Aaron J. Seigo
hi everyone... i just spent the last 36-ish hours flying from calgary to chicago to london where i had 3 hours of meetings (and an hour and a half of commuting) after which flew from london to helsinki in time to get to sleep at half past midnight or so. i'm up tomorrow at 06:30 and my last s

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

2008-10-27 Thread Dmitry Suzdalev
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.vidsolbach.de/r/245/ --- (Updated 2008-10-26 16:13:07.755803) Review request for Plasma and Aaron Se

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

2008-10-27 Thread Dmitry Suzdalev
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.vidsolbach.de/r/245/ --- (Updated 2008-10-26 16:11:43.080868) Review request for Plasma and Aaron Se

TaskManager::windowChangedGeometry() displacable?

2008-10-27 Thread christian mollekopf
I noticed in void TaskManager::windowChanged(WId w, unsigned int dirty) the windowChangedGeometry(t) signal isn't neccessary anymore and we can easely switch to the windowChanged(changes) signal. Should i change this or is this on purpose? This is the questionable code. TaskChanges chan

Re: Replacing Xesam runner with Nepomuk runner

2008-10-27 Thread Riccardo Iaconelli
On Monday 27 October 2008 19:32:02 Sebastian Kügler wrote: > I think replacing a known-broken runner with a working one is reasonable at > this stage. +1 -Riccardo -- GPG key: 3D0F6376 When encrypting, please encrypt also for this subkey: 9EBD7FE1 - Pace Peace Paix Paz Frieden Pax Pokój Frið

Re: Replacing Xesam runner with Nepomuk runner

2008-10-27 Thread Sebastian Kügler
On Monday 27 October 2008 17:10:20 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 > that we could make an exception here and see the Nepomuk runner as a fix to > the Xesam r

Re: tasks painting semplification

2008-10-27 Thread Marco Martin
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 really know how and where to fix them since > > i > > > don't know very well the libtaskmanager code: the issue of show only > > minimized tasks we ta

Re: [PROBLEM] New systray notifications and kwin

2008-10-27 Thread Chani
On October 27, 2008 02:51:17 Dmitry Suzdalev wrote: > On Wednesday 22 October 2008 19:26:22 Sebastian Kügler wrote: > > Only the ones that open popups automatically. The power management > > control extender from the battery and the calendar should have focus. > > (But I think it's fine to make tha

Re: tasks painting semplification

2008-10-27 Thread Chani
> however still speaking of the taskbar but on a totally different area: > there are some issues i don't really know how and where to fix them since i > don't know very well the libtaskmanager code: the issue of show only > minimized tasks we talked briefly on irc yesterday (sorry, didn't have mu

Re: tasks painting semplification

2008-10-27 Thread Marco Martin
On Monday 27 October 2008, christian mollekopf wrote: > > From: [EMAIL PROTECTED]> To: plasma-devel@kde.org> Subject: tasks > > painting semplification> Date: Mon, 27 Oct 2008 16:32:03 +0100> > Hi > > guys,> with Nuno we are starting to design the new plasma theme for 4.2, > > of course > this time

Replacing Xesam runner with Nepomuk runner

2008-10-27 Thread Sebastian Trüg
As mentioned in my other email I implemented a Nepomuk search runner. It is currently in playground/base/nepomuk-kde/krunner/search in case you want to give it a spin. I propose to replace the Xesam runner in kdebase with my nepomuk runner. Why? 1. The Xesam runner does not work since we have no

Re: tasks painting semplification

2008-10-27 Thread Jamboarder
> From: Marco Martin <[EMAIL PROTECTED]> > aaanyways, the painting function it's really a mess because when i added the > support for svg theming i decided to add a fallback for every element, so > when > it does not exist in the svg just paints a rounded rectangle, so the code is > 80% manual

[PATCH-API-DOCU] AbstractRunner documentation

2008-10-27 Thread Sebastian Trüg
Hi guys, while implementing the Nepomuk Search runner I stumbled over the incomplete, partially outdated, and partially wrong documentation of Plasma::AbstractRunner. I tried to fix it as far as I could from looking at the code. Please review the attached patch. The patch also contains a few co

RE: tasks painting semplification

2008-10-27 Thread christian mollekopf
> From: [EMAIL PROTECTED]> To: plasma-devel@kde.org> Subject: tasks painting > semplification> Date: Mon, 27 Oct 2008 16:32:03 +0100> > Hi guys,> with Nuno > we are starting to design the new plasma theme for 4.2, of course > this time > won't be a total redesign like for 4.1, but just some twe

tasks painting semplification

2008-10-27 Thread Marco Martin
Hi guys, with Nuno we are starting to design the new plasma theme for 4.2, of course this time won't be a total redesign like for 4.1, but just some tweaks here and there. a thing that however will look quite different is the taskbar theme, since there are some usability problems with the curren

Re: plasma apps and screen information

2008-10-27 Thread Guillaume Pothier
2008/10/27 Aaron J. Seigo <[EMAIL PROTECTED]>: >> - Added three methods to Corona: >>int numScreens() >>QRect screenGeometry(int) >>QRect availableScreenGeometry(int) > > these shouldn't be pure virtual. they can return silly values like by default, > of course: > ok > >> The latter s

Re: Taskbar with only one opened window

2008-10-27 Thread Rodrigo Pelorosso
Works like a charm now, you guys are the best, thanks a lot! On Sun, Oct 26, 2008 at 12:53 PM, Marco Martin <[EMAIL PROTECTED]> wrote: > On Saturday 25 October 2008, Aaron J. Seigo wrote: > > On Saturday 25 October 2008, Rodrigo Pelorosso wrote: > > > Sometimes it can be a little annoying, for w

Re: plasma apps and screen information

2008-10-27 Thread Aaron J. Seigo
On Friday 24 October 2008, Guillaume Pothier wrote: > 2008/10/23 Aaron J. Seigo <[EMAIL PROTECTED]>: > > On Wednesday 22 October 2008, Guillaume Pothier wrote: > >> Just tell me if you want me to go ahead. > > > > please do =) > > Ok, here comes the patch. In summary: > - Added three methods to Cor

Re: [PROBLEM] New systray notifications and kwin

2008-10-27 Thread Dmitry Suzdalev
On Wednesday 22 October 2008 19:26:22 Sebastian Kügler wrote: > Only the ones that open popups automatically. The power management control > extender from the battery and the calendar should have focus. (But I think > it's fine to make that explicit in the applet itself.) Ok, so we need some functi