Do not reply to this email. You can add comments to this bug at https://bugzilla.mozilla.org/show_bug.cgi?id=3512
John Daggett (:jtd) jdaggett@mozilla.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Attachment #556253|review?(jdaggett@mozilla.co |review+ Flags|m) |
--- Comment #78 from John Daggett (:jtd) jdaggett@mozilla.com 2011-09-04 18:59:44 PDT --- Comment on attachment 556253 --> https://bugzilla.mozilla.org/attachment.cgi?id=556253 (updated) patch to fix computation of StyleDistance to account for font-stretch more accurately
// TODO (refine CSS spec...): discuss priority of italic vs stretch // (currently, font-stretch takes precedence; not sure this is good)
One minor nit, the comment above no longer belongs here, it's really a todo for you to post on www-style if you think italic should take precedence over stretch.
When I revised the font matching algorithm, I made font-stretch higher priority because it has the greatest effect on layout. We discussed the font matching algorithm at the spring CSS F2F in Mountain View and there seemed to be general concurrence with this approach. I'm happy with discussing this again on www-style but I don't think a comment like this in the code is appropriate at this point.