On Mon, Jun 01, 2015 at 08:50:49PM +0200, Jürgen Spitzmüller wrote:
> 2015-06-01 19:34 GMT+02:00 Enrico Forestieri <for...@lyx.org>:
>
> > On Mon, Jun 01, 2015 at 03:50:49PM +0200, Enrico Forestieri wrote:
> > > On Mon, Jun 01, 2015 at 07:26:55AM +0100, Guillaume M-M wrote:
> > > >
> > > > 1) b3d2ffc0 introduces a regression in stable and master wrt 2.1.3.
> > See the
> > > > attached lyx-bug-font2.lyx. This is because legacy_lyxpreview2ppm.py
> > now
> > > > includes the font lmodern (L236). I do not understand this addition:
> > font
> > > > setting is already done in the preamble. Removing the line fixes the
> > issue.
> > >
> > > It was added in order to have vector fonts, otherwise the previews
> > > generated by the legacy method would be of very poor quality. It is
> > > difficult to determine what a user could do in the preamble and whether
> > > it is safe using lmodern or not. However, given that it may produce
> > issues,
> > > it may be better not using it, if nobody has a better idea.
> >
> > Jürgen, any idea on this one?
> >
>
> lmodern must only be loaded if no other font has been loaded, i.e. if
> computer modern is the active font:
>
> \def\tempa{cmr}
>
> \ifx\f@family\tempa\usepackage{lmodern}\else\fi
Thanks. I have applied a fix based on your suggestion.
--
Enrico
