-
Notifications
You must be signed in to change notification settings - Fork 320
Add reference file for font case #604
Conversation
chenxix
commented
Aug 21, 2014
Automatic validation checks of commit 4058628 passed. |
4058628 to
c8efab9
Compare
Automatic validation checks of commit c8efab9 passed. |
c8efab9 to
fddd33f
Compare
Automatic validation checks of commit fddd33f passed. |
fddd33f to
6a15582
Compare
Automatic validation checks of commit 6a15582 passed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lines 10~12 use the same technology to be tested. This is not a good reference file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This case is to check if the 'font' shorthand property can accept and set font-variant, font-size and font-family at the same time. So I set these 3 attributes one by one. Is there some other way to set the same font style ?
|
For the Ahem font usage, please refer to http://testthewebforward.org/docs/test-style-guidelines.html#special-fonts |
6a15582 to
473dafa
Compare
|
Reviewed 11 of 11 files at r1. css21/fonts/font-033-ref.html, line 7 [r1] (raw file): css21/fonts/font-044-ref.html, line 8 [r1] (raw file): Comments from the review on Reviewable.io |
|
Thank you for your PR. I landed the |