kdereview: events runner

2010-08-24 Thread Aaron J. Seigo
hi .. the events runner has been in kdereview for a while now and it is time to decide what to do with it :) Alexey: if there are no known issues with it, please move it into kdeplasma- addons/runners/ kdeplasma-addons has no hard restrictions on coding style, but we do encourage the use of kd

Re: kdereview: adjustable clock

2010-08-24 Thread Aaron J. Seigo
On Tuesday, August 24, 2010, Emdek wrote: > What specifically doesn't work properly? if i use the pre-built mode with the reflections in a panel of ~27px high, it doesn't fit. -- Aaron J. Seigo humru othro a kohnu se GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA EE75 D6B7 2EB1 A7F1 DB43 KDE core

kdreview: dictionary runner; changes needed to Plasma::RunnerManager?

2010-08-24 Thread Aaron J. Seigo
hi ... looking at the dictionary runner, i've noticed a few things that should be improved: * when the teardown() signal is emitted, the runner should disconnect from the DataEngine but i've also come to a few concerning conclusions: * a thread can wait indefinitely on m_wait.wait(&m_mutex);

Re: kdereview: adjustable clock

2010-08-24 Thread Emdek
Hello On 24-08-2010 at 21:45:01 Aaron J. Seigo wrote: > "adjustable clock" has been in kdereview for quite a while. it's time to > figure out where it goes. Finally. :-) > it still doesn't work properly in all sizes of panels, and i don't think > it > can ever be made to do so. What specifi

kdereview: adjustable clock

2010-08-24 Thread Aaron J. Seigo
hi... "adjustable clock" has been in kdereview for quite a while. it's time to figure out where it goes. it still doesn't work properly in all sizes of panels, and i don't think it can ever be made to do so. my suggestion therefore is to put it into extragear/plasma/ and ship it from there. i

Re: Review Request: Add "Show Icon only" option to the tasks applet

2010-08-24 Thread Aaron Seigo
> On 2010-08-23 09:00:48, Marco Martin wrote: > > this very patch appeared here for several times already. > > and as usual, the question is: what real value gives over auto hiding the > > text when there is not enough space? > > Markus Slopianka wrote: > If this patch works with the other

Re: Review Request: Add "Show Icon only" option to the tasks applet

2010-08-24 Thread Todd
> On 2010-08-23 09:00:48, Marco Martin wrote: > > this very patch appeared here for several times already. > > and as usual, the question is: what real value gives over auto hiding the > > text when there is not enough space? > > Markus Slopianka wrote: > If this patch works with the other

Re: Review Request: Add "Show Icon only" option to the tasks applet

2010-08-24 Thread Marco Martin
> On 2010-08-23 09:00:48, Marco Martin wrote: > > this very patch appeared here for several times already. > > and as usual, the question is: what real value gives over auto hiding the > > text when there is not enough space? > > Markus Slopianka wrote: > If this patch works with the other

Re: QtJolie and remote widgets

2010-08-24 Thread Kevin Ottens
On Sunday 22 August 2010 11:50:41 Kevin Ottens wrote: > On Saturday 21 August 2010 23:59:52 Dario Freddi wrote: > > You probably remember that back in the days (I'm talking about Tokamak in > > Randa) I helped Rob getting remote widgets in shape for release. This > > also implied putting a private

Re: Review Request: Add "Show Icon only" option to the tasks applet

2010-08-24 Thread Todd
> On 2010-08-23 09:00:48, Marco Martin wrote: > > this very patch appeared here for several times already. > > and as usual, the question is: what real value gives over auto hiding the > > text when there is not enough space? > > Markus Slopianka wrote: > If this patch works with the other

Re: Need assistance on bug #224666

2010-08-24 Thread Sharuzzaman Ahmat Raslan
Yes, I have cleared /var/tmp For 4.4.5 version in Debian, the error is still same. It was mentioned that the bug was fixed in trunk, but the bug report did not mention which commit number. Therefore I need to know the commit number so that I can forward that to Debian developers. Thanks.

Re: Review Request: Add "Show Icon only" option to the tasks applet

2010-08-24 Thread Beat Wolf
> On 2010-08-23 09:00:48, Marco Martin wrote: > > this very patch appeared here for several times already. > > and as usual, the question is: what real value gives over auto hiding the > > text when there is not enough space? > > Markus Slopianka wrote: > If this patch works with the other