Re: Re: Plasmate previewer, again =P

2009-09-11 Thread Diego Casella ([Po]lentino)
> > -- Messaggio inoltrato -- > From: "Aaron J. Seigo" > To: plasma-devel@kde.org > Date: Thu, 10 Sep 2009 16:53:48 -0600 > Subject: Re: Plasmate previewer, again =P > On September 10, 2009, Diego Casella ([Po]lentino) wrote: > > > -- Messaggio inoltrato -- > > > Fr

Re: Plasma-devel Digest, Vol 15, Issue 25

2009-09-11 Thread Diego Casella ([Po]lentino)
> > -- Messaggio inoltrato -- > From: Yuen Hoe Lim > To: plasma-devel@kde.org > Date: Fri, 11 Sep 2009 10:56:09 +0800 > Subject: Re: Plasmate previewer, again =P > >> * The previewer is awesome, but its possible to test it with a "fake" > >> package and see if it load it correctly

Re: Re: Plasmate previewer, again =P

2009-09-11 Thread Yuen Hoe Lim
> Right now previewer/test can be used to view installed applets. We are > unable to "execute" an applet currently. > > We can, at least for plasmoids :) The previewer class has a method that takes a system path, which 'executes' non-installed plasmoids from source. Jason "moofang" Lim Yuen

Re: Kickoff Krunner whitelist

2009-09-11 Thread Ivan Čukić
> 1. Keyboard navigation of the search results appears to have broken, > can you please have a look at that. The only thing changed is the data model. Do you have any idea how that could screw up the keybd nav? > 2. The search results for applications now take longer to appear than > they used

Re: Re: Re: Plasmate previewer, again =P

2009-09-11 Thread Diego Casella ([Po]lentino)
> > -- Messaggio inoltrato -- > From: Shantanu Tushar Jha > To: plasma-devel@kde.org > Date: Fri, 11 Sep 2009 09:53:37 +0530 > Subject: Re: Re: Plasmate previewer, again =P > > >>- Green flag: package signed by both KDE and the developer ( = >>completely trusted ); >>-

Re: Review Request: RunnerManager: Whitelist enforcement (bugfix)

2009-09-11 Thread Ivan Čukić
> so setAllowedPlugins probably needs to store its list in a separate key and > compare against it first. will commit a patch shortly. > OK, I'll cancel the review request ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/

Knowledgebase plasmoid in kdeplasma-addons

2009-09-11 Thread Marco Martin
Hi all, it's a while the knowledge base plasmoid (used to query the opendesktop.org knowledge base) is in kdereview. i plan to move it now in kdeplasmaaddons. is it ok? any objections? Cheers, Marco Martin ___ Plasma-devel mailing list Plasma-devel@kde.

Re: Review Request: big revamp of Device Notifier

2009-09-11 Thread Giulio Camuffo
> On 2009-09-10 18:18:27, Jacopo De Simoi wrote: > > /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/configurationpage.ui, > > line 63 > > > > > > I am not a native speaker either.. but this "the" seems strange

Re: Review Request: big revamp of Device Notifier

2009-09-11 Thread Michal Dutkiewicz
On 2009-09-10 18:18:27, Giulio Camuffo wrote: > > I still don't like the fact that you have to click twice to select the > > action if there is only one available action.. we should find a solution > > for this.. > > Giulio Camuffo wrote: > I agree. > Maybe we could show the actions on

Re: more info on the TaskGroupItem crash

2009-09-11 Thread David Faure
On Friday 11 September 2009, Aaron J. Seigo wrote: > On September 10, 2009, David Faure wrote: > > I finally took the time to investigate the plasma crash that was > > happening so often to me. > > can you try the attached patch? thanks. The patch fits exactly my analysis of the problem, but it l

Re: more info on the TaskGroupItem crash

2009-09-11 Thread David Faure
On Friday 11 September 2009, Aaron J. Seigo wrote: > On September 10, 2009, David Faure wrote: > > I finally took the time to investigate the plasma crash that was > > happening so often to me. > > can you try the attached patch? thanks. It definitely hits this code path often... QGraphicsScene:

Re: Stylesheets and plasma widgets

2009-09-11 Thread Marco Martin
On Friday 11 September 2009, Aaron J. Seigo wrote: > On September 10, 2009, alan moore wrote: > > Thanks; I tried this, but it didn't work (or else I'm doing it wrong). > > looking at Plasma::PushButton, it doesn't allow setting the font directly. > it uses the theme's font. it should probably not

Re: Review Request: big revamp of Device Notifier

2009-09-11 Thread Matthew Dawson
On Friday 11 September 2009 05:56:52 Giulio Camuffo wrote: > > > On 2009-09-10 18:18:27, Jacopo De Simoi wrote: > > > /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/configurationpage.ui, > > > line 63 > > > > > > >

Re: Review Request: big revamp of Device Notifier

2009-09-11 Thread Giulio Camuffo
> On 2009-09-10 18:18:27, Jacopo De Simoi wrote: > > /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/notifierview.cpp, > > line 194 > > > > > > May I ask why do you need to go backward? > > wrote: > Sinc

Re: Review Request: big revamp of Device Notifier

2009-09-11 Thread Michal Dutkiewicz
On 2009-09-10 18:18:27, Giulio Camuffo wrote: > > I still don't like the fact that you have to click twice to select the > > action if there is only one available action.. we should find a solution > > for this.. > > Giulio Camuffo wrote: > I agree. > Maybe we could show the actions on

Re: Review Request: big revamp of Device Notifier

2009-09-11 Thread Gilles CHAUVIN
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/1370/#review2289 --- /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/configurat

Re: Review Request: big revamp of Device Notifier

2009-09-11 Thread Aaron Seigo
On 2009-09-10 18:18:27, Giulio Camuffo wrote: > > I still don't like the fact that you have to click twice to select the > > action if there is only one available action.. we should find a solution > > for this.. > > Giulio Camuffo wrote: > I agree. > Maybe we could show the actions on

Re: Stylesheets and plasma widgets

2009-09-11 Thread Aaron J. Seigo
On September 11, 2009, Marco Martin wrote: > On Friday 11 September 2009, Aaron J. Seigo wrote: > > On September 10, 2009, alan moore wrote: > > > Thanks; I tried this, but it didn't work (or else I'm doing it wrong). > > > > looking at Plasma::PushButton, it doesn't allow setting the font > > dire

Re: Stylesheets and plasma widgets

2009-09-11 Thread Marco Martin
On Friday 11 September 2009, Aaron J. Seigo wrote: > On September 11, 2009, Marco Martin wrote: > > On Friday 11 September 2009, Aaron J. Seigo wrote: > > > On September 10, 2009, alan moore wrote: > > > > Thanks; I tried this, but it didn't work (or else I'm doing it > > > > wrong). > > > > > > lo

Re: Stylesheets and plasma widgets

2009-09-11 Thread Aaron J. Seigo
On September 11, 2009, Marco Martin wrote: > but the ones thatreally have to use the proxywidget to paint, like the ones > that have to layout text there wouldn't be much we can do about (maybe > syncing the nativewidget font with the plasma theme one?) that'd probably be the best we could do...

Re: Review Request: big revamp of Device Notifier

2009-09-11 Thread Aaron Seigo
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/1370/#review2298 --- is the latest version the most recent available version of the patch?

Re: Review Request: big revamp of Device Notifier

2009-09-11 Thread Aaron Seigo
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/1370/#review2300 --- /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/devicespac

Re: Review Request: big revamp of Device Notifier

2009-09-11 Thread Giulio Camuffo
> On 2009-09-11 19:47:12, Aaron Seigo wrote: > > /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/devicespaceinfodelegate.cpp, > > line 87 > > > > > > when is this not the case? i took that piece of code from p

Re: Review Request: big revamp of Device Notifier

2009-09-11 Thread Giulio Camuffo
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/1370/ --- (Updated 2009-09-11 21:32:17.946758) Review request for Plasma. Changes --

Re: Review Request: big revamp of Device Notifier

2009-09-11 Thread Aaron Seigo
> On 2009-09-11 19:47:12, Aaron Seigo wrote: > > /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/devicespaceinfodelegate.cpp, > > line 87 > > > > > > when is this not the case? > > wrote: > i took that pi

Review Request: forgetten slot defination

2009-09-11 Thread Omer F. USTA
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/1569/ --- Review request for Plasma and Aaron Seigo. Summary --- checkAvailability mu

Re: Review Request: forgetten slot defination

2009-09-11 Thread Omer F. USTA
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/1569/ --- (Updated 2009-09-11 22:10:15.009367) Review request for Plasma and Aaron Seigo.

Re: Review Request: forgetten slot defination

2009-09-11 Thread Aaron Seigo
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/1569/#review2305 --- Ship it! - Aaron On 2009-09-11 22:10:15, Omer F. USTA wrote: > > -

kwin animated show only for windows on all desktops?

2009-09-11 Thread Aaron J. Seigo
hi ... i just noticed that kwin's animated show effect only seems to work when i do: KWindowSystem::setOnAllDesktops(window->winId(), true); any particular reason for that? -- Aaron J. Seigo humru othro a kohnu se GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA EE75 D6B7 2EB1 A7F1 DB43 KDE