[Kst] extragear/graphics/kst/src/libkstapp

George Staikos staikos at kde.org
Sat Mar 18 23:23:09 CET 2006


I will revert this change if I don't hear back about it.  Is there no 
interest?

On Tuesday 14 March 2006 15:15, George Staikos wrote:
> I'm worried about this change for many reasons:
>
> 1) modified() causes many things to happen.  For instance, it sets the
> document to modified, so that exit prompts for save.  This is not what we
> want in these cases based on my last discussions with Barth about this.
> 2) It causes a forced update -> performance issue, and also if we're
> paused, we get an update when we shouldn't
> 3) It only covers a few cases.  KstScript that modifies the plot will not
> result in updated dialogs this way, for example.
>
> On Tuesday 14 March 2006 13:38, Andrew Walker wrote:
> > SVN commit 518631 by arwalker:
> >
> > BUG:123568 Ensure that plot dialog is sync'ed with plot
> >
> >  M  +2 -1      kst2dplot.cpp
> >
> >
> > --- trunk/extragear/graphics/kst/src/libkstapp/kst2dplot.cpp
> > #518630:518631 @@ -4174,6 +4174,7 @@
> >      kstdDebug() << "mouse release: do update" << endl;
> >      setDirty();
> >      static_cast<KstViewWidget*>(view)->paint();
> > +    emit modified();
> >    }
> >  }
> >
> > @@ -5054,7 +5055,7 @@
> >    if (handled) {
> >      view->paint();
> >      e->accept();
> > -    //emit modified();
> > +    emit modified();
> >      return;
> >    }
> >
> > _______________________________________________
> > Kst mailing list
> > Kst at kde.org
> > https://mail.kde.org/mailman/listinfo/kst

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


More information about the Kst mailing list