On Tue, Nov 19, 2002 at 11:27:14AM +0000, Angus Leeming wrote:

> >     121 void FormPreferences::ok()
> >     122 {
> >     123         FormBaseDeprecated::ok();
> >     124
> >     125 // FIXME !!
> >     126         if (colors_.modifiedXformsPrefs) {
> >     127                 string const filename =
> >     128                         AddName(user_lyxdir, "preferences.xform");
> >     129                 colors_.modifiedXformsPrefs =
> > !XformsColor::write(filename); 130         }
> >     131
> >     132         lv_.dispatch(FuncRequest(LFUN_SAVEPREFERENCES));
> >     133 }
> >
> > I need to save the file ... ideas ?
> >
> > john
> 
> ControlPreferences::apply() should invoke
>       view().apply();
> I think that it's up to the view to deal with this, don't you?

This is a change of behaviour, though. The above saves the xforms prefs
at "save", you're proposing to make it at apply() too. It's fine  by me,
but we should be aware it's a change ...

regards
john

-- 
Khendon's Law: If the same point is made twice by the same person,
the thread is over.

Reply via email to