Summary: | line-height in font shorthand does not override a previously stated line-height property | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Dan Richman <dan> | ||||||||
Component: | CSS | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | gavin.sharp | ||||||||
Priority: | P2 | Keywords: | HasReduction | ||||||||
Version: | 523.x (Safari 3) | ||||||||||
Hardware: | Mac | ||||||||||
OS: | OS X 10.4 | ||||||||||
URL: | http://level39.com/font-shorthand/ | ||||||||||
Attachments: |
|
Description
Dan Richman
2007-03-23 15:15:44 PDT
Created attachment 13787 [details]
Testcase
Created attachment 14641 [details]
First attempt
This should do it. It can be cleaned up a bit more I think, but I hope the approach is right.
Cheers,
Rob.
Comment on attachment 14641 [details]
First attempt
Per conversation on IRC, I suggested just always making line-height be mapped in as one of the first properties like font.
However instead of going into the RenderStyle in the switch statement it would just be cached as a member variable on the style selector.
After the style has been determined (e.g., around the time adjustRenderStyle is going to get called), that line height value (if it exists) should be mapped in for real.
Created attachment 14650 [details]
Different approach
As discussed with Hyatt on IRC, this is probably a better approach.
Cheers,
Rob.
Comment on attachment 14650 [details]
Different approach
r=me
Confirmed by existence of patches. |