Re: Cannot remove scripted plasmoid

2009-06-19 Thread Aaron J. Seigo
On Friday 19 June 2009, Petri Damstén wrote: > On Thursday 18 June 2009 22:23:58 Chani wrote: > > On June 18, 2009 09:15:27 Petri Damstén wrote: > > > On Saturday 13 June 2009 10:10:44 Petri Damstén wrote: > > > > > doh! that was in the wrong init() method (the private vs the public > > > > > one)

Re: Cannot remove scripted plasmoid

2009-06-19 Thread Chani
> > c++ plasmoids have been setting default size in their constructors, so > > it's probably best not to override that :) > > AppletPrivate::init is called from Plasma::Applet constructor so it gets > run before actual Applet constructor where resize usually is? I was > thinking that it could be u

Re: Cannot remove scripted plasmoid

2009-06-19 Thread Petri Damstén
On Thursday 18 June 2009 22:23:58 Chani wrote: > On June 18, 2009 09:15:27 Petri Damstén wrote: > > On Saturday 13 June 2009 10:10:44 Petri Damstén wrote: > > > > doh! that was in the wrong init() method (the private vs the public > > > > one) .. it's fixed now :) > > > > > > I think that X-Plasma-

Re: Cannot remove scripted plasmoid

2009-06-18 Thread Chani
On June 18, 2009 09:15:27 Petri Damstén wrote: > On Saturday 13 June 2009 10:10:44 Petri Damstén wrote: > > > doh! that was in the wrong init() method (the private vs the public > > > one) .. it's fixed now :) > > > > I think that X-Plasma-DefaultSize code should be still called from > > private in

Re: Cannot remove scripted plasmoid

2009-06-18 Thread Petri Damstén
On Saturday 13 June 2009 10:10:44 Petri Damstén wrote: > > doh! that was in the wrong init() method (the private vs the public one) > > .. it's fixed now :) > > I think that X-Plasma-DefaultSize code should be still called from private > init, since public init is too late for that? I was a way fo

Re: Cannot remove scripted plasmoid

2009-06-13 Thread Petri Damstén
On Friday 12 June 2009 18:21:30 Aaron J. Seigo wrote: > On Friday 12 June 2009, Petri Damstén wrote: > > On Thursday 11 June 2009 18:43:39 Aaron J. Seigo wrote: > > > great; we still need to fix the plasmoid (or the python scriptengine, > > > which is broken) but it's good to know we can handle the

Re: Cannot remove scripted plasmoid

2009-06-12 Thread Aaron J. Seigo
On Friday 12 June 2009, Petri Damstén wrote: > On Thursday 11 June 2009 18:43:39 Aaron J. Seigo wrote: > > great; we still need to fix the plasmoid (or the python scriptengine, > > which is broken) but it's good to know we can handle these situations > > with some robustness at least. > > Seems tha

Re: Cannot remove scripted plasmoid

2009-06-12 Thread Petri Damstén
On Thursday 11 June 2009 18:43:39 Aaron J. Seigo wrote: > great; we still need to fix the plasmoid (or the python scriptengine, which > is broken) but it's good to know we can handle these situations with some > robustness at least. Seems that the problem is in Plasma::AppletPrivate::setupScriptSu

Re: Cannot remove scripted plasmoid

2009-06-11 Thread Aaron J. Seigo
On Thursday 11 June 2009, Petri Damstén wrote: > On Wednesday 10 June 2009 20:30:36 Aaron J. Seigo wrote: > > On Wednesday 10 June 2009, Aaron J. Seigo wrote: > > > we should still catch such mistakes gracefully, however. does the > > > attached patch help? > > > > that patch could be hazardous to

Re: Cannot remove scripted plasmoid

2009-06-11 Thread Petri Damstén
On Wednesday 10 June 2009 20:30:36 Aaron J. Seigo wrote: > On Wednesday 10 June 2009, Aaron J. Seigo wrote: > > we should still catch such mistakes gracefully, however. does the > > attached patch help? > > that patch could be hazardous to your health. try the attached one instead > as it actually

Re: Cannot remove scripted plasmoid

2009-06-10 Thread Aaron J. Seigo
On Wednesday 10 June 2009, Aaron J. Seigo wrote: > we should still catch such mistakes gracefully, however. does the attached > patch help? that patch could be hazardous to your health. try the attached one instead as it actually works. i'm not sure i like adding that sort of overhead on every p

Re: Cannot remove scripted plasmoid

2009-06-10 Thread Aaron J. Seigo
On Wednesday 10 June 2009, Petri Damstén wrote: > Seems that this is because mainConfig is created without containment: > plasma-desktop(1574)/libplasma Plasma::AppletPrivate::mainConfigGroup: > requesting config for "Now Rocking" without a containment! which probably means that config() is being

Re: Cannot remove scripted plasmoid

2009-06-10 Thread Petri Damstén
On Friday 05 June 2009 09:05:58 Petri Damstén wrote: > On Thursday 04 June 2009 22:17:54 Aaron J. Seigo wrote: > > define "removing": removing from plasma by click on the "x" in the handle > > (or "Remove this..." in the context menu)? removing from disk with > > plasmapkg? removing via the add wid

Re: Cannot remove scripted plasmoid

2009-06-04 Thread Petri Damstén
On Thursday 04 June 2009 22:17:54 Aaron J. Seigo wrote: > define "removing": removing from plasma by click on the "x" in the handle > (or "Remove this..." in the context menu)? removing from disk with > plasmapkg? removing via the add widgets gui? . inquiring minds want to > know! ;) > > (if it

Re: Cannot remove scripted plasmoid

2009-06-04 Thread Aaron J. Seigo
On Thursday 04 June 2009, Petri Damstén wrote: > Played with python plasmoids again. When removed they seem to be getting > back at plasma restart. Then I tested with javascript plasmoid which also > comes back (At the same restart C++ plasmoid was removed correctly). Any > ideas what might be wron