<div dir="ltr"><div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Am Mo., 12. Apr. 2021 um 22:55 Uhr schrieb Jean-Marc Lasgouttes <<a href="mailto:lasgouttes@lyx.org" target="_blank">lasgouttes@lyx.org</a>>:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Le 12/04/2021 à 20:56, Jean-Marc Lasgouttes a écrit :<br>
> Le 10/04/2021 à 10:46, Enrico Forestieri a écrit :<br>
>> I think you reintroduced <a href="https://www.lyx.org/trac/ticket/10263" rel="noreferrer" target="_blank">https://www.lyx.org/trac/ticket/10263</a><br>
> <br>
> This is getting interesting. After getting half mad trying to understand <br>
> this issue, I tried something a bit radical: what about reverting the <br>
> fix 001f5a47 to ticket #10263?<br>
> <br>
> The answer is: nothing changes, this bug is still fixed. Something<br>
> <br>
> Could somebody confirm my hypothesis (see attachment)?<br>
<br>
The probable answer to this puzzle it the following commit, which was <br>
pushed 4 days after Enrico's commit:<br>
<br>
commit 812ff7de7550f7c5370584294e61d59d8bbb8700<br>
Author: Juergen Spitzmueller <<a href="mailto:spitz@lyx.org" target="_blank">spitz@lyx.org</a>><br>
Date:   Sun Aug 16 11:18:49 2020 +0200<br>
<br>
     Take out inheritFont() condition<br>
<br>
<br>
This was the right thing to do indeed, since inheritFont has nothing to <br>
do with the need to close fonts before an inset.<br></blockquote><div><br></div><div>Yes. If this works now: great. This font ahndling is so nastily complex that it always took me long to dive into it once more. <br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

I have to add that I do not understand half of the work that was done in <br>
this area in July/August 202 by you both, but the result seems very nice.<br>
<br>
OTOH, I suspect that the description of inheritFont is now misleading, <br>
and that it should be stripped down to something like:<br>
<br>
/**<br>
  * The font is inherited from the parent for LaTeX export if this<br>
  * method returns true, and the font is inherited for all other<br>
  * exports as well as on screen.<br>
  * If this method returns false the document default font is used<br>
  * inside the inset for all exports and on screen.<br>
  */<br>
virtual bool inheritFont() const { return true; }<br></blockquote><div><br></div><div>Good. If we can get rid of the "all other exports" part, it'd be perfect.</div><div><br></div><div>Thanks for investigating into our corssover menu.</div><div><br></div><div>Jürgen<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
<br>
JMarc<br>
<br>
-- <br>
lyx-devel mailing list<br>
<a href="mailto:lyx-devel@lists.lyx.org" target="_blank">lyx-devel@lists.lyx.org</a><br>
<a href="http://lists.lyx.org/mailman/listinfo/lyx-devel" rel="noreferrer" target="_blank">http://lists.lyx.org/mailman/listinfo/lyx-devel</a><br>
</blockquote></div></div>
</div>