Bug with line-height (font shorthand)

[David Hyatt]

Now I'm confused.  That's all I did do... called update() before 
applying the line-height portion of the font shorthand.... although 
because of:

style="font: 1em/1em Ahem; font-size:36px"

I think maybe this needs to be handled at a lower level by the parser 
(which needs to produce a separate line-height rule that could then be 
sorted and placed after the font rules).

dave

On Monday, February 24, 2003, at 07:59 AM, Lars Knoll wrote:

>
>> On Fre, 21 Feb 2003, David Hyatt wrote:
>>> font: 1em/1em Ahem;
>>
>> doh. can you confirm that the attached patch works?
>
> You're doing too much in that patch. There is no need to call update() 
> for the
> font size calculations, as these relate to the parent elements font. 
> Calling
> it before calculating the line height should be enough.
>
> Cheers,
> Lars
>
> _______________________________________________
> Khtml-devel@mail.kde.org
> http://mail.kde.org/mailman/listinfo/khtml-devel