RESOLVED FIXED 3231
CSS1: synthesize bold and oblique when those font variations are unavailable 
https://bugs.webkit.org/show_bug.cgi?id=3231
Summary CSS1: synthesize bold and oblique when those font variations are unavailable
Dave Hyatt
Reported 2005-06-01 14:38:00 PDT
If a bold version of a particular font is requested but none exists on the system, Web Kit does not properly synthesize a font. Instead it honors the weight but falls back to another font. Instead it should synthesize a font with the appropriate weight.
Attachments
add synthesized bold and oblique, other related renderer improvements (153.30 KB, patch)
2005-09-10 09:41 PDT, Darin Adler 		hyatt: review-
DetailsFormatted DiffDiff
revised patch fixing bug Mitz noticed, and with RTL back to rtl as requested (150.97 KB, patch)
2005-09-30 08:18 PDT, Darin Adler 		hyatt: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Nicholas Shanks
Comment 1 2005-06-07 07:32:36 PDT
If the font has an fvar weight axis, this can be used instead of synthesizing a heavier typeface. Possibly another bug should be opened to track this though, as fvar can be used for features other than just weight. Test case at http://web.nickshanks.com/safari/fvar
Dave Hyatt
Comment 2 2005-06-12 23:24:53 PDT
*** Bug 3494 has been marked as a duplicate of this bug. ***
Mark Rowe (bdash)
Comment 3 2005-09-08 21:17:38 PDT
Nick, the test case mentioned in comment #1 doesn't appear to be accessible at present. Would it be possible to attach it to the bug so that it will always be available?
Darin Adler
Comment 4 2005-09-10 09:39:31 PDT
Yes, the fvar possibility should be a separate bug.
Darin Adler
Comment 5 2005-09-10 09:41:04 PDT
Created attachment 3844 [details] add synthesized bold and oblique, other related renderer improvements
Darin Adler
Comment 6 2005-09-10 12:58:22 PDT
In Radar as <rdar://problem/3256269>.
mitz
Comment 7 2005-09-12 15:02:26 PDT
I think you have a problem in your order-swapping loop in _CG_drawRun:style:geometry:. You're decrementing end by 3 in each iteration.
Darin Adler
Comment 8 2005-09-12 17:50:27 PDT
Oops! You're right. I have to move that -- out! I'll have to make sure to run pixel tests rather than just layout tests. I'm leaving this patch in review? anyway so that Dave Hyatt can review it -- but he should review- it due to that bad mistake.
Dave Hyatt
Comment 9 2005-09-27 13:46:26 PDT
Comment on attachment 3844 [details] add synthesized bold and oblique, other related renderer improvements Just a personal preference, but I like style.rtl over style.RTL. Unless you feel really strongly, I'd rather this just stay the way it is currently. Looks good, but fix the mistake already pointed out, and I'll re-review.
Nicholas Shanks
Comment 10 2005-09-28 10:38:53 PDT
Opened bug #5168 to cover fvar
Darin Adler
Comment 11 2005-09-30 08:18:06 PDT
Created attachment 4106 [details] revised patch fixing bug Mitz noticed, and with RTL back to rtl as requested
Dave Hyatt
Comment 12 2005-10-03 14:44:56 PDT
Comment on attachment 4106 [details] revised patch fixing bug Mitz noticed, and with RTL back to rtl as requested r=me, make sure to run perf tests and layout tests to make sure we have no regressions.
Note You need to log in before you can comment on or make changes to this bug.