Re: Python Applet and configChanged

2009-05-27 Thread Petri Damstén
On Tuesday 26 May 2009 21:05:02 Aaron J. Seigo wrote: On Tuesday 26 May 2009, Petri Damstén wrote: Yes, seems to be called only once now. I think it still gets called when Cancel is pressed if configLoader == 0? this should be better in trunk now. testing with the javascript

Re: Python Applet and configChanged

2009-05-27 Thread Aaron J. Seigo
On Wednesday 27 May 2009, Petri Damstén wrote: On Tuesday 26 May 2009 21:05:02 Aaron J. Seigo wrote: On Tuesday 26 May 2009, Petri Damstén wrote: Yes, seems to be called only once now. I think it still gets called when Cancel is pressed if configLoader == 0? this should be better in

Python Applet and configChanged

2009-05-26 Thread Petri Damstén
with python applet start (configChanged called 4 times): main.py __init__ main.py init main.py constraintsEvent main.py constraintsEvent main.py constraintsEvent plasmoidviewer(18498) KCoreConfigSkeleton::writeConfig: plasmoidviewer(18498)/libplasma Plasma::Applet::configChanged: KConfigSkeleton

Re: Python Applet and configChanged

2009-05-26 Thread Aaron J. Seigo
On Tuesday 26 May 2009, Petri Damstén wrote: plasmoidviewer with python applet start (configChanged called 4 times): hm... too many calls to writeConfig are happening, and KConfigSkeleton is Too Dumb(tm) to know that nothing's actually changed. i'll take a look in a bit about this. (i don't

Re: Python Applet and configChanged

2009-05-26 Thread Petri Damstén
On Tuesday 26 May 2009 16:19:39 Aaron J. Seigo wrote: right, we probably don't need the applet to also say the config has changed if (but only if!) we have a config loader, so something like this in AppletPrivate::configDialogFinished(): if (!configLoader) { // manually call

Re: Python Applet and configChanged

2009-05-26 Thread Aaron J. Seigo
On Tuesday 26 May 2009, Petri Damstén wrote: Yes, seems to be called only once now. I think it still gets called when Cancel is pressed if configLoader == 0? this should be better in trunk now. testing with the javascript Plasma::ScriptEngine i'm only getting configChanged calls when the