[webkit-reviews] review requested: [Bug 6484] font-weight does not
properly support graded weights : [Attachment 12882] patch v 0.9
bugzilla-request-daemon at macosforge.org
bugzilla-request-daemon at macosforge.org
Fri Feb 2 11:12:40 PST 2007
Nicholas Shanks <contact at nickshanks.com> has asked for review:
Bug 6484: font-weight does not properly support graded weights
http://bugs.webkit.org/show_bug.cgi?id=6484
Attachment 12882: patch v 0.9
http://bugs.webkit.org/attachment.cgi?id=12882&action=edit
------- Additional Comments from Nicholas Shanks <contact at nickshanks.com>
working patch for feedback. addresses every issue mentioned on either my or
alex's patches.
does not include tests or changelog yet
Issues:
1) I am not sure if all my "using namespace" and "@namespace" changes are
necessary
2) The FontManager <=> CSS weight conversion table needs tweaking (or something
else does) since "normal" gives a Light weight and "bold" gives an Extra Bold
weight. This is not considered significant for the purposes of the review
request.
3) Does not assert on NSBoldFontMask being passed in. Trait is ignored anyway,
and no longer used elsewhere so I don't see it necessary to assert (other
ignored traits don't assert, for example).
More information about the webkit-reviews
mailing list