[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