[Kst] [Bug 111114] Ability to apply only settings that were changed in plot dialog

George Staikos staikos at kde.org
Tue Oct 25 00:31:51 CEST 2005


On Monday 24 October 2005 18:00, Barth Netterfield wrote:
> On October 24, 2005 05:42 pm, George Staikos wrote:
> > On Monday 24 October 2005 13:02, Barth Netterfield wrote:
> > > What do people think of making the plot dialog work the same as the
> > > data object dialogs: it keeps the number of new ways of doing things
> > > down, while offering maximum flexability.
> >
> >   In which sense?  I don't quite follow...
>
> -Use the 'edit multiple' paradim.
> -Have a 'create plot' and an 'edit plot' realization of the dialogs with
> 'ok', and 'cancel'.
> -get rid of 'apply to this window', 'apply to all windows', etc...'

  That would definitely help.

> Maybe we should wait on this until post 1.2, except for the next point, if
> it applies.

  Yes please.

> > > Code wise, it is not a lot harder than the current scheme.
> >
> >   Actually, on a side note, I still need to rework the data dialogs
> > entirely. They just plain crash with KstScript and the only workaround is
> > to go back to using Qt methods for everything (instead of showI, etc).
>
> I would like to understand this....

  In the script code, you would typically do this:
var dd = Kst.getElementByName("KstPsdDialog");  // a reference to the data 
dialog
dd->show();
Click around in the dialog a bit -> crash.  The dialog expects to be called 
with show_I() so is in an inconsistent state.

  This is a systematic problem, not an isolated crash.  We need to make our 
interfaces safe and consistent.

#7  0x402e19f8 in KstPsdDialogI::toggleEditMultiple (this=0x83e3918)
    at dataobjectdialog.h:142
#8  0x402e7b4b in KstPsdDialogI::qt_invoke (this=0x83e3918, _id=65, 
    _o=0xbfffe5d0) at kstpsddialog_i.moc:133
#9  0x4190b753 in QObject::activate_signal (this=0x8402d78, clist=0x841ae50, 

-- 
George Staikos
KDE Developer				http://www.kde.org/
Staikos Computing Services Inc.		http://www.staikos.net/


More information about the Kst mailing list