Bug 4227

Summary: The ex unit doesn't work for font-variant: small-caps
Product: WebKit Reporter: James Sumners <james.sumners>
Component: CSSAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, mitz
Priority: P2 Keywords: HasReduction
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
Testcase
none
Add font-variant to the list of properties applied first darin: review+

James Sumners
Reported 2005-07-30 15:45:19 PDT
p#test { font-variant: small-caps; letter-spacing: 0.2ex; } will produce a paragraph element with the text rendered in small caps but with normal letter spacing. p#test { letter-spacing: 0.2ex; font-variant: small-caps; } will produce a paragraph element with the text rendered in small caps and the letter spacing set to 0.2ex.
Attachments
Testcase (183 bytes, text/html)
2005-12-29 06:42 PST, Joost de Valk (AlthA) 		no flags
Details
Add font-variant to the list of properties applied first (4.96 KB, patch)
2006-02-17 04:28 PST, mitz 		darin: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Joost de Valk (AlthA)
Comment 1 2005-12-29 06:41:36 PST
Confirmed, testcase coming up.
Joost de Valk (AlthA)
Comment 2 2005-12-29 06:42:20 PST
Created attachment 5351 [details] Testcase
mitz
Comment 3 2006-02-17 01:27:03 PST
The problem seems to be only with the 'ex' unit. Specifying letter-spacing in other units works for small-caps. On the other hand, it is not limited to the letter-spacing property.
mitz
Comment 4 2006-02-17 04:28:42 PST
Created attachment 6562 [details] Add font-variant to the list of properties applied first
Darin Adler
Comment 5 2006-02-17 07:51:06 PST
Comment on attachment 6562 [details] Add font-variant to the list of properties applied first r=me
Alexey Proskuryakov
Comment 6 2006-02-19 09:04:54 PST
The test from this patch doesn't pass for me - there is no green box, and an error is reported: ================= ERROR: called xHeight on an empty QFontMetrics (/Users/ap/WebKit/WebCore/kwq/KWQFontMetrics.mm:153 xHeight) =================
Alexey Proskuryakov
Comment 7 2006-02-19 09:44:45 PST
Sorry, it was my mistake.
Note You need to log in before you can comment on or make changes to this bug.