Yes I missed it, thanks.

The non-scaling is intentional, as explained in the external-templates comment. 
Fonts are scaled to the surrounding text (unlike with XFig), a property that 
allowing rescaling here would destroy. Rescaling should rather be done inside 
Inkscape, by actually rescaling the figure. But sure, it could be enabled and 
the responsibility left  to the user.

About the preview inset, I think I just don't understand the logic here well 
enough (and I don't use preview as it's heavy on the CPU with large docs), but 
if you can make the necessary change (and if it's simple) just make it and go 
ahead. 

Yes, I would like to see this checked in.

Best, Martin
________________________________________
From: Guillaume Munch [g...@lyx.org]
Sent: Wednesday, September 28, 2016 1:45 AM
To: Vermeer Martin
Subject: Fwd: Re: Inkscape /LyX integration (working!)

Hi, I am forwarding to you this message sent to the lyx-devel list last
week in case you missed it.


-------- Message transféré --------
Sujet : Re: Inkscape /LyX integration (working!)
Date : Wed, 21 Sep 2016 23:51:13 +0200
De : Guillaume Munch <g...@lyx.org>
Pour : lyx-devel@lists.lyx.org
Groupes de discussion : gmane.editors.lyx.devel
Références :
<4a4a9e56a391a1479fbe21e7590e45fcac501...@exmdb07.org.aalto.fi>
<20160831223720.ga9...@atrey.karlin.mff.cuni.cz>
<4a4a9e56a391a1479fbe21e7590e45fc010c6ae...@exmdb07.org.aalto.fi>
<4a4a9e56a391a1479fbe21e7590e45fc010c6b0...@exmdb07.org.aalto.fi>
<20160905183545.gc9...@atrey.karlin.mff.cuni.cz>
<4a4a9e56a391a1479fbe21e7590e45fc010c6b1...@exmdb07.org.aalto.fi>

Le 06/09/2016 à 05:48, Vermeer Martin a écrit :
> Pavel thanks!
>
> Attached the changes you proposed (though still to HEAD).
>
> Yes, the Inkscape bug is known upstream. Fixing seems to be a bit tricky as 
> it is due to a change in the Cairo library, aimed at enabling the output of 
> "layers", transparent or not, covering or covered by text strings. But of 
> course Inkscape should know itself how many PDF pages it is outputting...
>
> Someone please push if this passes muster
>


Hi Martin,

I have rebased your patches against master and it works.

One thing I have noticed is that the preview provided by the external
inset does not update when one changes the surrounding text. For
instance the preview can fail because of a missing package and it will
not update after correcting the preamble, or it will not reflect the
surrounding font. In practice it works much better to wrap the
ExternalInset in a preview inset, which takes the surroundings into
account for updating the preview.

Also, is it possible to enable the resizing options from the dialog?

Do you want me to commit the patches?


Guillaume



Reply via email to