>>>>> "Alfredo" == Alfredo Braunstein <[EMAIL PROTECTED]> writes:

>>>> Inheriting is the right thing to do in tabulars, for example.
     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Alfredo> You didn't say that before.

Are you sure?

>> I guess InsetNote and InsetBranch may need that too, but I am not
>> sure.

Alfredo> So what's the right fix? You seem to understand this much
Alfredo> better than I do.

I do not know actually. I do not have a good understanding of what
1.3.x did.

Setting the default font from the context just before computing
metrics is reasonable, since the fonts contained in a paragraph are
always relative to a base font. Therefore, it does not seem outrageous
to me to set this value in the metrics call. However, as you point
out, this is not valid for all insets. Would a good solution be to put
this code in the metrics() method of InsetTabular (that is, set the
font of all cells)?

I'll try to test your patch.

JMarc

Reply via email to