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

George Staikos staikos at kde.org
Tue Mar 14 21:15:38 CET 2006


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