On Fri, May 06, 2005 at 10:55:16AM +0100, Angus Leeming wrote:
> Juergen Spitzmueller wrote:
>
> > In LyXFunc::dispatch, case LFUN_INSET_APPLY, there's an update call
> > missing. You see it when you apply changes from an inset dialog. You have
> > to do some keyboard action until the changes get
On Fri, May 06, 2005 at 11:55:29AM +0200, Juergen Spitzmueller wrote:
> In LyXFunc::dispatch, case LFUN_INSET_APPLY, there's an update call missing.
> You see it when you apply changes from an inset dialog. You have to do some
> keyboard action until the changes get visible.
>
> The attached pat
Georg Baum wrote:
> I think that Jürgens patch is fine, I would however add a comment stating
> that in an ideal world 'update' would be set by the inset.
I did so and committed.
Jürgen
Angus Leeming wrote:
> I'm not saying it's wrong, but I note that 'update' is specified in very
> few places in this function. Do all insets change there visible appearance
> on LFUN_INSET_APPLY or should this 'update' flag be set by indvidual
> insets.
>
> Just thinking out aloud.
In theory not
Juergen Spitzmueller wrote:
> In LyXFunc::dispatch, case LFUN_INSET_APPLY, there's an update call
> missing. You see it when you apply changes from an inset dialog. You have
> to do some keyboard action until the changes get visible.
>
> The attached patch adds the update call.
>
> OK?
>
> Jürg
