RFR: 8191758: Match WebKit's font weight rendering with JavaFX
Phil Race
prr at openjdk.java.net
Fri Apr 17 18:08:16 UTC 2020
On Thu, 16 Apr 2020 12:40:24 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:
>> As per JavaFx 700 font weight is considered to be bold but webkit is using 600 font weight for text to become bold. to
>> fix issue, use boldWeightValue() function which uses 700 font weight rather than isFontWeightBold() which compare
>> against 600 font weight.
>
> Can you add a unit test to go along with this fix?
Per the opentype spec, 700 is bold. 600 is semi-bold
https://docs.microsoft.com/en-us/typography/opentype/spec/os2#usweightclass
CSS agrees : https://developer.mozilla.org/en-US/docs/Web/CSS/font-weight
So are you saying webkit has been using bold at a lower weight than these specs suggest ?
I see the logic all comes from Source/WebCore/platform/graphics/FontSelectionAlgorithm.h
I suppose the existing code thinks that if we have reached what that file calls the bold threshold of 600 then we
should use bold. It isn't necessarily "wrong" but I think I agree that it is more important to be consistent with the
rest of Java FX ... which I believe is the point of this change ?
-------------
PR: https://git.openjdk.java.net/jfx/pull/180
More information about the openjfx-dev
mailing list